Skip to content

Conversation

@story645
Copy link
Contributor

Added timezone offsets because running into same problem as #732

Added timezone offsets because running into same problem as scientific-python#732
@netlify
Copy link

netlify bot commented May 15, 2025

Deploy Preview for scientific-python-org ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 15e3aea
🔍 Latest deploy log https://app.netlify.com/projects/scientific-python-org/deploys/68264c9c266729000894cc2f
😎 Deploy Preview https://deploy-preview-764--scientific-python-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 91
Accessibility: 100
Best Practices: 100
SEO: 100
PWA: -
View the detailed breakdown and full score reports

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

@story645 story645 marked this pull request as ready for review May 15, 2025 20:20
@bsipocz
Copy link
Member

bsipocz commented May 15, 2025

Thanks!

@bsipocz bsipocz merged commit 48cdb02 into scientific-python:main May 15, 2025
4 of 5 checks passed
@story645 story645 deleted the patch-2 branch May 15, 2025 20:45
@story645
Copy link
Contributor Author

hmm, still not working properly - do I also have to remove the timezone at the top or does the reingest take time?

@bsipocz
Copy link
Member

bsipocz commented May 15, 2025

Unfortunately, I think the only way this would work if you have UTC times. You will have to shift manually around twice a year if you want to keep the same local timeslot at least until this problem persist in the tooling.

story645 added a commit to story645/scientific-python.org that referenced this pull request May 15, 2025
bsipocz pushed a commit that referenced this pull request May 16, 2025
* remove named timezone from mpl file

Follow up to  #764,

* converted timestamps to utc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants