Social links redirection issue in social block in footer.
This only for two social links
  1. Instagram URL
  2. LinkedIn URL

After installation you will find the block in footer

Link should be redirect into social site.

CommentFileSizeAuthor
#11 after_patch.png289.35 KBtanuja bohra
#7 Screenshot 2023-08-08 at 4.22.06 PM.png841.86 KBdjsagar
#6 issue_after_patch.png21.57 KBtanuja bohra
#6 issue.png403.15 KBtanuja bohra
#2 Screenshot 2023-08-03 at 5.23.02 PM.png179.02 KBdjsagar
#2 Screenshot 2023-08-03 at 5.21.16 PM.png1.62 MBdjsagar
Bug.png1.32 MBdjsagar

Issue fork nature_trail-3379010

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

djsagar created an issue. See original summary.

djsagar’s picture

djsagar commented
Issue summary: View changes
StatusFileSize
new Screenshot 2023-08-03 at 5.21.16 PM.png1.62 MB
new Screenshot 2023-08-03 at 5.23.02 PM.png179.02 KB

djsagar opened merge request !2

djsagar’s picture

djsagar commented
Status: Active » Needs review
Jaspreet-Kaur’s picture

Jaspreet-Kaur commented

MR! 2 Resolve the hyperlink issue. Thanks djsagar!

tanuja bohra’s picture

tanuja bohra commented
Status: Needs review » Needs work
StatusFileSize
new issue.png403.15 KB
new issue_after_patch.png21.57 KB

@djsagar, I can still see the issue for linkdin. Added the screenshot for reference

djsagar’s picture

djsagar commented
Issue summary: View changes
StatusFileSize
new Screenshot 2023-08-08 at 4.22.06 PM.png841.86 KB

@Tanuja Bohra,
Patch will change the ; to :, as i can see your screen-short the issue was resolved.

instagram_url: https;//instagram.com => instagram_url: 'https://instagram.com

linkedin_url: htpps;//linkedin.com' =>linkedin_url: 'htpps://linkedin.com'

Please check one more time and let me know if there is any changes require.

Thanks!

djsagar’s picture

djsagar commented
Status: Needs work » Needs review
tanuja bohra’s picture

tanuja bohra commented

@djsagar , yes, but the title of the issue is regarding the social link redirection and still the linkdin url has wrong spelled "htpps"
can we fix that?

djsagar’s picture

djsagar commented

@Tanuja Bohra Thanks for correction updated MR.

tanuja bohra’s picture

tanuja bohra commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new after_patch.png289.35 KB

thanks @djsagar , issue is resolved.

Bhupendra_Raykhere made their first commit to this issue’s fork.

bhupendra_raykhere’s picture

bhupendra_raykhere commented
Status: Reviewed & tested by the community » Fixed
bhupendra_raykhere’s picture

bhupendra_raykhere commented
Status: Fixed » Closed (fixed)