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

Feature #107 And Added Bottom Footer #128

Merged
merged 5 commits into from
Nov 10, 2024

Conversation

sarinsanyal
Copy link
Contributor

On the basis of #107, I added the Bottom Footer and the Social Icons (with only GitHub and LinkedIn Working).

Also added a Newsletter Subscribing implementation and "Terms and Conditions", "Privacy Policy", etc. links that don't redirect to anywhere as of now.

At the end, I put in Acknowledgements for the creators and the Contributors.

Bottom.Navbar.and.Footer.mp4

@Amna-Hassan04 Please take a look at this video, and let me know if everything is alright. Thank you!

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better.

Please follow these guidelines for smooth collaboration:

  1. After completing a task, add your name and details to the Contributors Table/List.
  2. Attach relevant screenshots or videos of your contribution for transparency.
  3. Tasks are assigned on a first-come, first-served basis.
    • Avoid spamming requests for already-assigned tasks.
    • Respect the queue and wait for new tasks.
  4. Propose New Ideas: Discuss new features or suggestions in the "Ideas" channel before starting work.

@sarinsanyal
Copy link
Contributor Author

@Amna-Hassan04 If you think this is alright, can you please merge the commit? Thank you once again.

Copy link
Contributor Author

@sarinsanyal sarinsanyal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the Newsletter UI and the Script Code, according to the changes prescribed. Based on #107 and #108 (comment)

@Amna-Hassan04 Please let me know if it is alright now. Also please add the GSSoC and Hacktoberfest Labels. Thank you!

@Amna-Hassan04
Copy link
Owner

@sarinsanyal plz do make sure your pr uses the latest version of the project so i can merge

@sarinsanyal
Copy link
Contributor Author

I have resolved the conflict @Amna-Hassan04. Please check and let me know if it's alright.

@Amna-Hassan04
Copy link
Owner

@sarinsanyal plz confirm that this PR is using the latest version of the project so i can merge.

@zoobiasyed
Copy link
Collaborator

zoobiasyed commented Oct 29, 2024

@sarinsanyal kindly update your pr to the latest version so that Amna can merge it. As the test already mentions that it has failed in having the pr up to date. Kindly do it asap. Otherwise everything looks good.

@sarinsanyal
Copy link
Contributor Author

@Amna-Hassan04 @zoobiasyed I have merged it once again. Please let me know if I can help you in any other way. Thank you.

@zoobiasyed
Copy link
Collaborator

@Amna-Hassan04 I think it's good to go now. Thank you @sarinsanyal.

@Amna-Hassan04 Amna-Hassan04 merged commit d514c05 into Amna-Hassan04:main Nov 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants