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

Security page added #1474

Merged
merged 2 commits into from
Aug 1, 2024
Merged

Conversation

jency1
Copy link
Contributor

@jency1 jency1 commented Jul 25, 2024

Related Issue

#1446

Description

Security page added

Type of PR

  • Feature enhancement
  • Documentation update

Screenshots / videos (if applicable)

[Attach any relevant screenshots or videos demonstrating the changes]

Screenshot 2024-07-25 113502

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Copy link

vercel bot commented Jul 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flipkart-clone ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 30, 2024 6:54pm

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.

Great job, @jency1! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.We will promptly review your changes and offer feedback. kindly remember to check our contributing guidelines

@jency1
Copy link
Contributor Author

jency1 commented Jul 26, 2024

@arghadipmanna101 Please review and merge this PR.

@jency1
Copy link
Contributor Author

jency1 commented Jul 27, 2024

@arghadipmanna101 Please review and merge this PR as soon as possible. I have made all the required changes which was assigned by you.

1 similar comment
@jency1
Copy link
Contributor Author

jency1 commented Jul 29, 2024

@arghadipmanna101 Please review and merge this PR as soon as possible. I have made all the required changes which was assigned by you.

@jency1
Copy link
Contributor Author

jency1 commented Jul 30, 2024

@arghadipmanna101 You have assigned this issue and I had completed it within 1 day and you are not merging it. As you have assigned this issue, you must merge this PR.

Currently, the security link in the footer section will not redirect you to security page by clicking on security in footer links.

I have created the new page as well as linked it with the footer so that by clinking on the security in the footer you will be redirected to the security page.

Please merge this PR as soon as possible.

@jency1
Copy link
Contributor Author

jency1 commented Jul 30, 2024

@arghadipmanna101 I have solved the conflict issues. Please, merge this PR.

@arghadipmanna101 arghadipmanna101 merged commit c309ed2 into arghadipmanna101:main Aug 1, 2024
2 checks passed
@arghadipmanna101 arghadipmanna101 added level3 An Hard issue gssoc Fun with GSSOC'24 labels Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc Fun with GSSOC'24 level3 An Hard issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants