DDMAL / CantusDB

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

Combine feasts: 2456 -> 4474 and 2094 -> 4475 #1560

Open dchiller opened 1 month ago

dchiller commented 1 month ago

We need to:

lucasmarchd01 commented 1 month ago

I think we can set up a management command that can do this. There's one thing I've noticed:

The feast date is different for feasts 2456 and 4474, which are August 8 and August 4, respectively. Should we use the date from the old feast (2456) or the new one (4474) when merging the data?

annamorphism commented 1 month ago

My understanding is that the merged feast (4474) should have the 4 August date, corresponding to the new code 14080430.

dchiller commented 1 month ago

I think we can set up a management command that can do this.

Great point -- I could see this as being potentially useful functionality going forward.

lucasmarchd01 commented 4 weeks ago

The command has now been run successfully on staging. Once this reaches production we can run the command again and close the issue.

dchiller commented 4 weeks ago

@lucasmarchd01 Thanks! I assigned you mostly so that we could see from the issue list that this was already under control -- but thanks for testing this too!