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

#8651 [mB] Kiezradar Backend: Updated "Kiez Hinzufügen" (Radius) in the Kiezauswahl - Error Messages missing for invalid location pin and too many radiuses #5992

Open
sblindow opened this issue Jan 28, 2025 · 2 comments
Labels
Prio: Low fix when there is time Type: Bug incorrect or uexpected behaviour

Comments

@sblindow
Copy link
Contributor

sblindow commented Jan 28, 2025

URL:
/django-admin

user:
Admin

expected behaviour:
When admins fill out the form at /meinberlin_kiezradar/kiezradar/add/, they should see error messages if they enter invalid information.

behaviour:
The system fails to show error messages in two cases:

  1. When trying to enter locations outside of Berlin
  2. When attempting to add a sixth radius to a user profile

Instead of showing proper error messages, the system crashes and displays a "500 Internal Server Error."

Note:
Error messages are working correctly for all other types of invalid entries.

Screenshot:

Image
@sblindow sblindow changed the title #8651 [mB] Kiezradar Backend: Updated "Kiez Hinzufügen" (Radius) in the Kiezauswahl - Error Messages missing for impermissible location pin and too many radiuses #8651 [mB] Kiezradar Backend: Updated "Kiez Hinzufügen" (Radius) in the Kiezauswahl - Error Messages missing for invalid location pin and too many radiuses Jan 28, 2025
@CarolingerSeilchenspringer CarolingerSeilchenspringer added Type: Bug incorrect or uexpected behaviour Prio: Medium fixed and released with next scheduled release labels Jan 30, 2025
@goapunk
Copy link
Contributor

goapunk commented Feb 26, 2025

Is this functionality we need (adding a kiez from the django admin)? Otherwise we could just disable it which is probably much quicker to fix

@CarolingerSeilchenspringer
Copy link
Contributor

@goapunk @sblindow this is correct, I don't see the need to add a Kiez via django as this is a very user centered functionality, so let's disable.

@CarolingerSeilchenspringer CarolingerSeilchenspringer added Prio: Low fix when there is time and removed Prio: Medium fixed and released with next scheduled release labels Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prio: Low fix when there is time Type: Bug incorrect or uexpected behaviour
Projects
None yet
Development

No branches or pull requests

3 participants