You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently under beta, but soon to be released, is a contact form. It will allow logged in users to messages workspaces. Right now if a user abuses the messaging feature, the only option is for admins to delete the user account.
We want to change it so that a user must be authenticated through the sign-up journey AND is not blocked from sending messages.
Build suggestion
Add a utility for both UserContactForm on the frontend and the isValidEmailCreationRequest function that checks a user has verified their email address and does not have the isBlockedFromMessaging set to true on their account.
For the form, it's fine just to keep the form hidden if the conditions aren't met.
Testing
At a minimum please include these new scenarios in the UserContactForm test script. But you'll probably also create a little util so add a test for that too. Plus I'd personally love it if a cypress test is added for this too.
The text was updated successfully, but these errors were encountered:
Description
Currently under beta, but soon to be released, is a contact form. It will allow logged in users to messages workspaces. Right now if a user abuses the messaging feature, the only option is for admins to delete the user account.
We want to change it so that a user must be authenticated through the sign-up journey AND is not blocked from sending messages.
Build suggestion
Add a utility for both
UserContactForm
on the frontend and theisValidEmailCreationRequest
function that checks a user has verified their email address and does not have theisBlockedFromMessaging
set to true on their account.For the form, it's fine just to keep the form hidden if the conditions aren't met.
Testing
At a minimum please include these new scenarios in the
UserContactForm
test script. But you'll probably also create a little util so add a test for that too. Plus I'd personally love it if a cypress test is added for this too.The text was updated successfully, but these errors were encountered: