Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

✨ Skip user details step for logged in users #602

Merged
merged 9 commits into from
Mar 23, 2023
Merged

Conversation

lukevella
Copy link
Owner

When a user is logged in, we no longer ask for their name and email since this is provided during registration.

This also contains some significant changes to how notifications work. Most notably:

  • Only logged in users can toggle notifications on and off
  • Notifications are no longer restricted to the email address that was used to create the poll
  • Anyone with access to an admin page can toggle notifications for their user

@vercel
Copy link

vercel bot commented Mar 23, 2023

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

Name Status Preview Comments Updated
web ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 23, 2023 at 0:12AM (UTC)

@lukevella lukevella merged commit d8e3dcd into main Mar 23, 2023
@lukevella lukevella deleted the remove-3rd-step branch March 23, 2023 12:18
jonas-hoebenreich pushed a commit to jonas-hoebenreich/rallly that referenced this pull request Mar 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant