Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linkedin Icon added in social media icons in every page #3101

Closed
wants to merge 23 commits into from

Conversation

pinakviramgama
Copy link
Contributor

πŸ› οΈ Fixes Issue

Fixes: #3055

πŸ‘¨β€πŸ’» Description

What does this PR do?

The linkedin icon in social media section was not present, but now it's added in every file.

here's the output of it.

image

Copy link

vercel bot commented Nov 9, 2024

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
fin-veda βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Nov 9, 2024 4:52pm

Copy link

github-actions bot commented Nov 9, 2024

Thank you for submitting your chaotic pull request! πŸ™Œ We'll review it as soon as possible. In the meantime, please ensure that your changes align with our chaotic CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@pinakviramgama
Copy link
Contributor Author

Dear @deepeshmlgupta , please merge this PR.

Copy link

netlify bot commented Nov 9, 2024

βœ… Deploy Preview for getfinveda ready!

Name Link
πŸ”¨ Latest commit cbac83a
πŸ” Latest deploy log https://app.netlify.com/sites/getfinveda/deploys/672f92f819a4010008c8cfe2
😎 Deploy Preview https://deploy-preview-3101--getfinveda.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 39
Accessibility: 66
Best Practices: 75
SEO: 73
PWA: 50
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@sampadatiwari30
Copy link
Collaborator

@pinakviramgama blog.html solve this conflict.

@pinakviramgama
Copy link
Contributor Author

Dear @sampadatiwari30 , conflict is resolved, please merge this PR.

@pinakviramgama
Copy link
Contributor Author

Dear @sampadatiwari30 , please merge this PR

@pinakviramgama
Copy link
Contributor Author

Dear @deepeshmlgupta , please merge this PR

@deepeshmlgupta
Copy link
Collaborator

deepeshmlgupta commented Nov 9, 2024

@pinakviramgama firstly you have to remove the unnecessary files that are not needed for modification for this PR then only this PR will merge
One more thing why have you added more and more blank lines in the codebase

@deepeshmlgupta
Copy link
Collaborator

image
what is the meaning of doing this

@pinakviramgama
Copy link
Contributor Author

Sorry sir @deepeshmlgupta for that blank spaces , it's by mistake

@sampadatiwari30
Copy link
Collaborator

@pinakviramgama blog.html
maps.html solve this conflict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FEATURE: The Linkedin Icon is missing in social media icons in footer
3 participants