Skip to content

Conversation

dmytrotsko
Copy link
Contributor

Combined covidcast and flu, etc. sections.

Copy link

netlify bot commented Apr 25, 2024

Deploy Preview for cmu-delphi-main ready!

Name Link
🔨 Latest commit 9ab22ad
🔍 Latest deploy log https://app.netlify.com/sites/cmu-delphi-main/deploys/666086d3fe6cc100088c01da
😎 Deploy Preview https://deploy-preview-962--cmu-delphi-main.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dmytrotsko dmytrotsko marked this pull request as draft April 25, 2024 20:23
@dmytrotsko dmytrotsko marked this pull request as ready for review April 25, 2024 20:43
@melange396
Copy link
Contributor

Update: @dmytrotsko was looking into adding aliases for the new paths, so old URLs will still work. We should keep the aliases around at least until search engines recrawl our site and update the links in their indexes.

@nmdefries
Copy link
Contributor

Approved by Roni

@dmytrotsko dmytrotsko requested a review from dshemetov June 5, 2024 15:40
Copy link
Collaborator

@dshemetov dshemetov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

COVID and Flu & Other Pathogens are now combined in one section as requested in OKRS24-24. The /epidemic-signals/_index.md also has the alias covid19, for backwards compatibility. Footer looks good.

@dshemetov dshemetov merged commit 16cfbe4 into dev Jun 5, 2024
@dshemetov dshemetov deleted the OKRS24-24 branch June 5, 2024 17:50
@melange396 melange396 mentioned this pull request Jun 7, 2024
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.

4 participants