DDMAL / CantusDB

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

some(?) hymn verse IDs causing chant-create issues (again?) #1553

Closed annamorphism closed 3 days ago

annamorphism commented 3 weeks ago

This may be another case of #1515, but I wanted to make note of it... The ID 008367:01 seems to cause havoc for creating any subsequent chants. Other verses of the same hymn--or verses of other hymns--don't seem to be a problem, just this one...

lucasmarchd01 commented 4 days ago

I can't seem to replicate the issue. What were the steps involved to produce the error?

I tried creating a new source and adding a chant with CID 008367:01, then when I add any subsequent chants it works as expected.

annamorphism commented 4 days ago

Strange...when I add a chant to my test source, using the ID 008367:01 and a dummy text (blablabla) I get image

It makes it into the inventory though: image

But adding a new chant: 500 error image

Deleting this chant and adding a new one with some other ID in the same way works as expected.

annamorphism commented 4 days ago

tried it on a different test source, just in case it was something about the source. Same thing: image

annamorphism commented 4 days ago

Made a brand-new source, too, and the issue still happens...?? I'm consistently able to replicate the problem... image

lucasmarchd01 commented 4 days ago

Interesting...

Okay, I was able to replicate it now on the production site. On staging and develop, there doesn't seem to be any issues. Maybe something in the pipeline has already fixed this? I'll go and see if I can try to pinpoint which commit might be responsible for it.

We're updating production soon, so we can see if it's been resolved then.

lucasmarchd01 commented 4 days ago

I believe https://github.com/DDMAL/CantusDB/pull/1517 should fix this issue, but we should test to make sure once it makes it to production!

dchiller commented 3 days ago

This is now working on production with the 15July2024 update.