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

Raise PermissionDenied if anon user tries to access create_password #4371

Merged
merged 3 commits into from
Feb 3, 2025

Conversation

frjo
Copy link
Member

@frjo frjo commented Feb 3, 2025

Also:

  • Add some missing user string translations.
  • Update locale.
  • Remove Tamil translation since it contains a number of formatting errors.

Copy link

sentry-io bot commented Feb 3, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: hypha/apply/users/views.py

Function Unhandled Issue
create_password NotImplementedError: Django doesn't provide a DB representation for AnonymousUser. ...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

@frjo frjo added Type: Maintenance Type: Patch Mini change, used in release drafter labels Feb 3, 2025
…l translation since it contains a number of formatting errors.
@frjo frjo force-pushed the fix/user-string-translations branch from 53f7e74 to 9fdf991 Compare February 3, 2025 09:09
@frjo frjo changed the title Add some missing user string translations. Update locale. Raise PermissionDenied if anon user tries to access create_password Feb 3, 2025
@frjo frjo merged commit 00ff332 into main Feb 3, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant