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

Internal Server Error when Updating some contacts #3453

Open
Tracked by #2955
juliankehne opened this issue Feb 19, 2025 · 2 comments
Open
Tracked by #2955

Internal Server Error when Updating some contacts #3453

juliankehne opened this issue Feb 19, 2025 · 2 comments
Labels
bug Something isn't working needs-further-investigation This issue needs some further research/investigation

Comments

@juliankehne
Copy link

juliankehne commented Feb 19, 2025

Describe the Bug

I don't know the cause yet, but when updating some contacts on cms-test it returns a 500 Internal Server Error.

All 'broken' contacts seem to be inserted into one or more pages, but not all contacts inserted into a page are 'broken'.

Some 'broken' contacts:
https://cms-test.integreat-app.de/testumgebung/contact/34/edit/
https://cms-test.integreat-app.de/testumgebung/contact/32/edit/ (fixed)
https://cms-test.integreat-app.de/testumgebung/contact/29/edit/ (fixed)
https://cms-test.integreat-app.de/testumgebung/contact/22/edit/ (fixed)

Some of the contacts are fixed after removing the from the content they were inserted in.

Updating a page/location/poi with one of the broken contacts in it leads to another Internal Server Error.

Inserting one of the broken contacts in a new page works as expected.

Steps to Reproduce

  1. Open one of the contacts above
  2. Click 'Update'
  3. See Error

Expected Behavior

No error.

Actual Behavior

Error.

Additional Information

Video showing the behaviour: (if you cannot play it in github, right click it & 'save video as...')

414745555-8175848a-70bf-4a56-9bfe-bb6e732924ce.mp4
Traceback

Related Issues

@juliankehne juliankehne added bug Something isn't working needs-further-investigation This issue needs some further research/investigation labels Feb 19, 2025
@juliankehne
Copy link
Author

Updated description with more info and a video.

@juliankehne juliankehne mentioned this issue Feb 19, 2025
11 tasks
@charludo
Copy link
Contributor

Dang, that's not good. Possibly related: "old" contact card insertion seem to not be updated when changing contact details. "New" ones do, though.

Might be related? (Even though this is once more an issue I cannot reproduce locally... 🙈)

THe question is though: if this really only affects old test data, do we simply accept that something broke the old contacts/cards in the last couple PRs and just fix our test data? Since no prod data contains contacts yet, and all contacts/cards the regions will use are "new" ones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-further-investigation This issue needs some further research/investigation
Projects
None yet
Development

No branches or pull requests

2 participants