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

Signup #73

Merged
merged 9 commits into from
Apr 20, 2024
Merged

Signup #73

merged 9 commits into from
Apr 20, 2024

Conversation

bhaettasch
Copy link
Collaborator

@bhaettasch bhaettasch commented Nov 1, 2022

Create internal area on homepage and implement self-signup and related features.

This implements #56, #57 and prepares #69.

Introduce mixin for highlighting of current page in navbar
Additionally, remove obsolete imports in dachor_website/views
Adjust to color scheme
Adapt URLs
Add link to internal pages to navbar for authenticated users
Create base template for internal views
Adapt settings to redirect to start view of internal app if no target was specified
Add django-betterforms as dependency to handle whole signup process in one form, see https://django-betterforms.readthedocs.io/en/latest/multiform.html#working-with-createview
Create forms for user information and profile including custom required fields
Add views for signup and to inform about missing confirmation
Link signup form in login view
This implements #56
@bhaettasch bhaettasch marked this pull request as ready for review March 15, 2023 09:31
@bhaettasch bhaettasch requested a review from geislern March 15, 2023 09:31
A mail will be sent when a new user registered (this implements #57)
Prepare mail sending by introducing required settings for dev and production, including entries in the matching secrets sample file (this finishes the implementation parts of #69)
@geislern geislern merged commit 7862b4b into main Apr 20, 2024
2 checks passed
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.

2 participants