Skip to content

fix: require verified email to invite #66948

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

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mdtro
Copy link
Member

@mdtro mdtro commented Mar 14, 2024

Frontend support for: #66861

image

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 14, 2024
@mdtro mdtro force-pushed the mdtro/require-verified-email-to-invite-fe branch from 22cefee to c01c5fc Compare March 14, 2024 11:58
@mdtro mdtro marked this pull request as draft April 3, 2024 00:07
@getsantry
Copy link
Contributor

getsantry bot commented Apr 24, 2024

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you add the label WIP, I will leave it alone unless WIP is removed ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added the Stale label Apr 24, 2024
@mdtro mdtro added WIP and removed Stale labels Apr 24, 2024
@mdtro mdtro force-pushed the mdtro/require-verified-email-to-invite-fe branch from bf45572 to cb3d00d Compare April 29, 2024 16:53
@mdtro mdtro force-pushed the mdtro/require-verified-email-to-invite-fe branch from 345acba to 4620c62 Compare April 29, 2024 23:33
@mdtro mdtro force-pushed the mdtro/require-verified-email-to-invite-fe branch from 4620c62 to f8d0dba Compare April 29, 2024 23:34
@mdtro mdtro force-pushed the mdtro/require-verified-email-to-invite-fe branch from aeb7391 to f6dd77f Compare April 29, 2024 23:37
) : (
{!isVerified && !isSuperUser ? (
<Alert type="warning" showIcon>
{t('Please verify your email before inviting other users.')}
Copy link
Member

Choose a reason for hiding this comment

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

we have some very old users with unverified email so they won't have verify email in their inbox. We should add a button that would trigger that email again here in the flow.

Should we make this required? It seems to me that we should have emails verified and this would be a good way to enforce it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components WIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants