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

Merge Staging into Production, 2023-07-24 edition #881

Merged
merged 20 commits into from Jul 25, 2023
Merged

Conversation

jacobdgm
Copy link
Contributor

@jacobdgm jacobdgm commented Jul 24, 2023

Pages/views that have been modified:

  • admin area
    • feast list: a search box has been added to the top of the page, allowing filtering by name and feast_code
    • provenance list: a search box has been added to the top of the page, allowing filtering by name
    • sequence list: a search box has been added to the top of the page, allowing filtering by incipit, title and cantus_id
  • Full Index page
    • has been optimized. Other than loading faster, there should be no changes to how it looks/behaves
  • all pages
    • SSHRC logo is now clickable, and takes you to the SSHRC website (previously, it wasn't a link)

Other changes:

  • a restart policy has been added to docker-compose.yml. This means that if one of our servers goes down for some reason (as appears to have happened on staging over the weekend), it will start back up right away.
  • the sync scripts have been updated to the version we used while doing the final sync. We shouldn't need to run these again, but in the future we may want to know what they did, so now they're up-to-date
  • The project's CSRF settings have been updated, meaning users will be able to log in on Production (this change had already been applied as a hotfix last Friday)

jacobdgm and others added 20 commits July 17, 2023 14:47
add search fields to admin model pages
In footer, add link to SSHRC logo
Add restart policy to docker-compose.yml
Add cantusdatabase.org to CSRF_TRUSTED_ORIGINS
Use `select_related` to optimize "view full index" page of a source.
merge develop into staging, 2023-07-24 edition
@jacobdgm jacobdgm marked this pull request as ready for review July 24, 2023 20:12
@jacobdgm jacobdgm requested a review from JoyfulGen July 24, 2023 20:12
@JoyfulGen
Copy link
Contributor

@jacobdgm just a heads-up, you still have test manuscripts on https://cantusdatabase.org/, like this one:

My favourite manuscript

I usually type "hello" in search boxes to see what happens if I search for something that won't exist... imagine my surprise when I actually got a result! I know you're probably aware of tests like these being on there, but my surprise at finding a chant called "Hi I'm a chant" was too good not to share.

@jacobdgm
Copy link
Contributor Author

Ha! After consulting with @annamorphism, I actually just deleted all the test sources from Production earlier this morning (but evidently after you found this).

So long as you only saw this while you were logged-in and not while you were logged-out, @JoyfulGen, then everything is working as it should.

@jacobdgm jacobdgm merged commit af465e5 into production Jul 25, 2023
@jacobdgm
Copy link
Contributor Author

note: we're waiting to put these changes up on the Production machine until a couple of changes applied as hotfixes can be properly integrated into the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants