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

feat: added protection to feedback route #201

Merged
merged 3 commits into from
Jan 30, 2025

Conversation

ARYPROGRAMMER
Copy link
Contributor

Fixes Issue🛠️

Fixes a Part of Issue #137

Description👨‍💻

This PR fixes adds route protection to feedback page and necessary logics

Type of Change📄

  • Bug fix (non-breaking change which fixes a bug)
  • Style (non-breaking change which improves website style or formatting)

Checklist✅

  • I am an Open Source contributor
  • I have performed a self-review of my code
  • My code follows the style guidelines of this project
  • I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

demo.mp4

Copy link

vercel bot commented Jan 30, 2025

@ARYPROGRAMMER is attempting to deploy a commit to the Pratik0112's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Hey @ARYPROGRAMMER, Welcome to 💖TelMedSphere !!! 🎊
Thanks for raising a PR! Your effort makes this project better. 🙌
Please wait for the PR to be reviewed.
Happy Coding!! ✨

@PratikMane0112
Copy link
Owner

PratikMane0112 commented Jan 30, 2025

Rather to show a "Access Denied" Model you just redirect users to login form, it looks little bit fishy !!!!

Fixed it same as mentioned in #144

@ARYPROGRAMMER
Copy link
Contributor Author

@PratikMane0112 lgtm

@PratikMane0112 PratikMane0112 merged commit 328d6d2 into PratikMane0112:master Jan 30, 2025
0 of 2 checks passed
Copy link

@ARYPROGRAMMER Congrats, Your pull request has been successfully merged 🥳🎉
Thank you for your contribution to 💖TelMedSphere !!!
Happy coding 🎊, Keep Contributing 🙌 !!!

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.

2 participants