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

Content: Adding lots of new stories for the 10/8 NMNH Opening #129

Merged
merged 10 commits into from
Oct 7, 2024

Conversation

agurvich
Copy link
Collaborator

@agurvich agurvich commented Sep 25, 2024

Adding content

This is a big content update, so I want to merge into develop first rather than directly into main, despite our previously agreed upon procedure.

  • NMNH Location Page
  • New Location Features
    • SI Hyperwall
    • Eyes on Earth (for both HQ and NMNH)
    • Hometown Dashboard (for both HQ and NMNH)
    • Unveiling Time

Content checklist

Check the Content Documentation for more details.

Any content

  • Cover image with proper attribution (when relevant) is present.
  • Cover images do not contain any text (they're used as a background and text may not be readable).
  • All images are not too big/heavy. As a general rule of thumb, they should not weigh more than 500KB, or be larger than 2000px. See the media guide for more info.
  • Content was added using the available MDX Blocks.

Datasets

Stories

Copy link

netlify bot commented Sep 25, 2024

Deploy Preview for earth-information-center ready!

Name Link
🔨 Latest commit af99890
🔍 Latest deploy log https://app.netlify.com/sites/earth-information-center/deploys/67044eed89462f0008b63680
😎 Deploy Preview https://deploy-preview-129--earth-information-center.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.

@agurvich agurvich self-assigned this Sep 25, 2024
@agurvich agurvich marked this pull request as draft September 25, 2024 11:03
@agurvich
Copy link
Collaborator Author

agurvich commented Oct 7, 2024

@freitagb I cherry-picked out your commits from #131 . We should merge #131 and then push that into main before merging this into develop.

@agurvich agurvich marked this pull request as ready for review October 7, 2024 21:17
Copy link

@freitagb freitagb left a comment

Choose a reason for hiding this comment

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

looks good to me

@agurvich agurvich merged commit 65ec418 into develop Oct 7, 2024
6 checks passed
@agurvich agurvich deleted the stories/NMNH branch October 7, 2024 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants