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] Add warning for email verification status and allow to resend verification email #281

Open
nemesifier opened this issue Jun 17, 2024 · 2 comments · May be fixed by #333 or #336
Open

[feature] Add warning for email verification status and allow to resend verification email #281

nemesifier opened this issue Jun 17, 2024 · 2 comments · May be fixed by #333 or #336
Labels
enhancement New feature or request

Comments

@nemesifier
Copy link
Member

nemesifier commented Jun 17, 2024

If a user is accessing the admin and has email notifications enabled, we should check whether the user has its email verified.
If the email is not verified, we can show a warning message using the django message framework, something like:

Email notifications are enabled for your account, but since your email address has not been verified, email sending is currently disabled. Please verify your email address to enable email notifications.

We should link to a view that triggers the resending of the verification email.

We could use the user_logged_in signal to trigger this message, we should execute the receiver function only if the user is a staff user.

@nemesifier nemesifier added the enhancement New feature or request label Jun 17, 2024
@nemesifier nemesifier moved this to To do (general) in OpenWISP Contributor's Board Aug 21, 2024
@pandafy pandafy moved this from To do (general) to To do (Python & Django) in OpenWISP Contributor's Board Jan 13, 2025
SitaGanesh added a commit to SitaGanesh/openwisp-notifications that referenced this issue Feb 5, 2025
…send verification email openwisp#281

- Added a warning message for users with email notifications enabled but not verified.
- Linked to a view that triggers the resending of the verification email.
- Used the user_logged_in signal to trigger this message for staff users only.
- Created UserProfile model to track email verification status.
- Updated views to handle email verification and resending verification emails.
- Defined URL patterns for the new views.
@SitaGanesh SitaGanesh linked a pull request Feb 5, 2025 that will close this issue
4 tasks
@SitaGanesh
Copy link

Sir @nemesifier,

Could you please review the feature I have added locally? Let me know if it is working as expected or if any modifications are needed. If everything looks good, kindly merge this pull request. If there are any issues, please create a new issue, and I will work on addressing them.

It took me a considerable amount of time and effort to complete this feature, and I would be grateful if it could be merged.

Thank you.

@nemesifier
Copy link
Member Author

Sir @nemesifier,

Could you please review the feature I have added locally? Let me know if it is working as expected or if any modifications are needed. If everything looks good, kindly merge this pull request. If there are any issues, please create a new issue, and I will work on addressing them.

It took me a considerable amount of time and effort to complete this feature, and I would be grateful if it could be merged.

Thank you.

@SitaGanesh #333 (review), thanks!

SitaGanesh added a commit to SitaGanesh/openwisp-notifications that referenced this issue Feb 26, 2025
@SitaGanesh SitaGanesh linked a pull request Feb 26, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: To do (Python & Django)
2 participants