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: activation is the new registration #1892

Conversation

tomudding
Copy link
Member

@tomudding tomudding commented Aug 24, 2024

Description

Activation is the new registration. Furthermore, sync frequency increased as I will be doing less and less for GEWISWEB.

Related issues/external references

Fixes GH-1891. See also GEWIS/gewisdb#431.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation improvement (no changes to code)
  • Other (please specify)

Adds a notice to the login page during August and September to make it clear
that activation (and setting a password) is required to be able to use the
website.

Furthermore, migrates fully to the `activate` endpoint to not cause any
confusion. NOTE: this requires complementary changes in GEWISDB.
Because some people may not yet have activated their account on the website we
should temporarily keep the original URL active.
Because people cannot understand "tomorrow".

This does make me think, because in all other places we do not sync if
`sync_paused` is reported via the API. Maybe we should make an API call before
we sync?
@tomudding tomudding changed the title Feature/account activation instead of registration feat: activation is the new registration Aug 24, 2024
@tomudding tomudding merged commit e0f3d3c into GEWIS:main Aug 24, 2024
3 of 4 checks passed
github-actions bot added a commit that referenced this pull request Aug 24, 2024
Tom Udding: Merge pull request #1892 from tomudding/feature/account-activation-instead-of-registration

feat: activation is the new registration

Co-authored-by: tomudding <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change "register" to "activate" for new accounts
1 participant