-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feat: add press release "Four Northern California..." #246
Feat: add press release "Four Northern California..." #246
Conversation
✅ Deploy Preview for cal-itp-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
...o-buy-contactless-open-loop-fare-payment-systems-off-of-californias-purchasing-agreements.md
Show resolved
Hide resolved
...o-buy-contactless-open-loop-fare-payment-systems-off-of-californias-purchasing-agreements.md
Show resolved
Hide resolved
...o-buy-contactless-open-loop-fare-payment-systems-off-of-californias-purchasing-agreements.md
Show resolved
Hide resolved
...o-buy-contactless-open-loop-fare-payment-systems-off-of-californias-purchasing-agreements.md
Show resolved
Hide resolved
Fixed an issue where the post's datetime was in the future, so the page wasn't showing up |
@angela-tran I got an alert on this and wasn't sure what you meant here (today is 8/18/23 ... did I goof a date somewhere? thanks for helping me learn!). |
@esquared415 Oh, no the date in your source document is fine. The incorrect datetime was entirely on my end and has been fixed - I wrote that comment above just for my future self and other developers working on the site. Here's a direct link to the press release on the preview site: https://deploy-preview-246--cal-itp-website.netlify.app/press/four-northern-california-transit-agencies-join-forces-to-buy-contactless-open-loop-fare-payment-systems-off-of-californias-purchasing-agreements |
There was a problem hiding this 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! Let's wait until @esquared415 approves the preview.
Homepage
Press Page
With the filter pill selected
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@angela-tran @thekaveman This looks sooo good as a webpage instead of a standalone PDF—I love it! Real website content!! 📰 🥳
One Q for @mrose914 (but nothing that should hold us up from publishing): While I can see "Aug 2023" under the headline on the "Press" page, I don't see a date field on the press release itself—so if someone shares the direct link to the press release (bypassing the Press page), they'd have no sense of how recently this was published.
I think it would be worth having that month / year field appear somewhere on the press releases as well.
Thanks for considering, and please pub awayyyy!
Closes #245