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-Aug-02 edition #922

Merged
merged 25 commits into from Aug 4, 2023
Merged

Conversation

jacobdgm
Copy link
Contributor

@jacobdgm jacobdgm commented Aug 2, 2023

Pre-testing tasks:

  • update the staging server so we can test the new code - done!
  • run tests
  • ensure password reset feature works - it works!

General/global Changes:

  • Our containers automatically restart if they go down for some reason
  • All of our pages used to tell crawlers not to index them, meaning we were telling Google not to include these pages in search results. This has been removed, so Google can now start crawling the website!
  • When you hover your cursor over a link, the link now becomes underlined while remaining the same colour. (previously, links would only change colour (very, very slightly!) when hovered over)

Pages where changes took place:

  • Admin Area, Source Edit page
    • the "Title" field is much wider now
  • Chant Create
    • When the most recently created chant in the source has a Cantus ID of 909000, we don't try to generate a list of suggested chants
  • Chant Detail
    • Concordances summary: previously, the initialism for Portuguese Early Music Database read "MMMO". It now reads "PEM"
  • Chant Search
    • There was an extra/unnecessary space near the bottom of each cell that has been removed. This space has been removed.
  • Content Overview
    • There are now different tabs for each kind of object in the database, allowing project managers to view all recently-changed chants separately from all recently-changed sources, etc.
    • 100 results are now displayed per page (rather than 50), and results are paginated, allowing more to be loaded if necessary

jacobdgm and others added 25 commits July 31, 2023 17:13
Re-add "restart: always" to docker-compose.yml
Fix Portuguese Early Music initialism in Concordances Summary
Remove `<meta content="noindex">` from templates
Merge develop into staging, 2023-Aug-01 edition
Improve formatting of Chant Search page
content overview: add tabs for models and display all objects ordered by date created
Chant Create: prevent getting suggested chants for Cantus ID of 909000
admin: increase source title widget length
Merge develop into staging, 2022-Aug-02 edition
add content overview to permissions test suite
Merge develop into staging, 2023-Aug-02 bonus edition
@JoyfulGen
Copy link
Contributor

@jacobdgm I've been messing around trying to see if cantusdatabase.org shows up in Google search results and I'm not sure which of my findings are relevant, so here they all are:

  • When I search for "cantus database," the first result is OldCantus. NewCantus appears on the second page of results, and it's the "login for contributors" page, not the home page.
  • When I search for a specific manuscript siglum, like CH-E 611 Einsiedeln, again OldCantus appears first and NewCantus appears way lower. Also the NewCantus page refers to a specific folio, not the main Einsie page.
  • When I copied a sentence directly from the website ("Sixteenth-century antiphoner from the Premonstratensian abbey of Tongerlo (Belgium)," from https://cantusdatabase.org/source/702611) NewCantus didn't show up in the results at all. The page did appear when I added "site:cantusdatabase.org" to the search query.
  • A similar situation happened when I copied this sentence from the home page: "Cantus is a database of the Latin chants found in manuscripts and early printed books, primarily from medieval Europe." Except that here, when I forced it to search within cantusdatabase.org, Google told me no results matched my search.
  • There was also this: cantusdatabase.org/description does not exist, but appears on Google search #929

Please let me know if there's any of this I should investigate further! I hope it's useful.

@jacobdgm
Copy link
Contributor Author

jacobdgm commented Aug 3, 2023

Thanks for looking into this, @JoyfulGen! We can probably expect Google to update its suggestions over time - its crawlers don't work instantaneously. And once cantus.uwaterloo.ca starts to redirect traffic to cantusdatabase.org, the links to the OldCantus site will probably start to be replaced with links to NewCantus. The instruction to crawlers is not a thing that will suddenly make google link to the new site, but rather it will allow Google to do its thing in its own time.

@JoyfulGen
Copy link
Contributor

@jacobdgm that explains it, thank you! It all looks good then :)

@jacobdgm jacobdgm merged commit 272dbac into production Aug 4, 2023
@jacobdgm
Copy link
Contributor Author

jacobdgm commented Aug 4, 2023

This new code is running on staging! The password reset feature works properly. All tests are passing! Hooray!

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