DDMAL / CantusDB

A new site for Cantus Database running under Django.
https://cantusdatabase.org
MIT License
5 stars 6 forks source link

Error on staging: Reverse for 'institutionidentifier-detail' not found #1608

Open dchiller opened 2 months ago

dchiller commented 2 months ago

The staging Django container gives a django.urls.exceptions.NoReverseMatch exception. This seems to be because the BaseModel defines a get_absolute_url function that assumes that all models have a "detail" view. But the InstitutionIdentifier model does not have a detail view.

ahankinson commented 2 months ago

How did you trigger that? The institution identifier model isn’t meant to be a url endpoint

dchiller commented 2 months ago

To be honest, I can't recreate it. I was looking through the logs on the staging site trying to pinpoint the cause of another issue and saw this. There's nowhere that I can find in the code thus far that would try to call the get_absolute_url method on the InstitutionIdentifier model, so... at the very least, it seems like that method should allow for the possibility that the model in question won't have a detail view.

ahankinson commented 2 months ago

Do you have a full traceback?

dchiller commented 2 months ago

Do you have a full traceback?

Unfortunately, no.

dchiller commented 2 months ago

I'm going to put this on hold until we can recreate.