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

Added Icons #738

Closed
wants to merge 5 commits into from
Closed

Added Icons #738

wants to merge 5 commits into from

Conversation

sailaja-adapa
Copy link

@sailaja-adapa sailaja-adapa commented Jun 22, 2024

Hello Project Admin
I Added Icons to Navbar and Footer - Solved #704
Kindly Please go through it @thestarsahil
I solved the issue and conflict please check it and see if it is still exists

hh

Copy link

netlify bot commented Jun 22, 2024

👷 Deploy request for counsellor-startup pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 1555d12

Copy link

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

Copy link
Contributor

deepsource-io bot commented Jun 22, 2024

Here's the code health analysis summary for commits 0771e0a..1555d12. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 22 occurences introduced
🎯 40 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@sailaja-adapa
Copy link
Author

@thestarsahil Kindly please go through this PR

@thestarsahil
Copy link
Collaborator

@sailaja-adapa Due to this , whole pages effecting

@sailaja-adapa
Copy link
Author

@sailaja-adapa Due to this , whole pages effecting

is it? but it is working fine in my system

@thestarsahil
Copy link
Collaborator

Compare the difference
The localhost and Live host

@sailaja-adapa
Copy link
Author

@thestarsahil i think i solved that issue still it causing that problem?

@sailaja-adapa
Copy link
Author

Compare the difference The localhost and Live host

yeah okk @thestarsahil

@thestarsahil
Copy link
Collaborator

Yes !

@sailaja-adapa
Copy link
Author

i solved conflicts just now

@sailaja-adapa
Copy link
Author

okk then i will close this then @thestarsahil

@thestarsahil
Copy link
Collaborator

What i am trying to say, Just pull all the latest changes and make your branch uptodate according to master branch and then push it

@sailaja-adapa
Copy link
Author

What i am trying to say, Just pull all the latest changes and make your branch uptodate according to master branch and then push it

Yeah okk I will try it once @thestarsahil

@sailaja-adapa sailaja-adapa closed this by deleting the head repository Jun 23, 2024
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.

None yet

2 participants