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

Fixes Calendar not working offline #139

Merged
merged 1 commit into from
Sep 21, 2024
Merged

Conversation

Spoffy
Copy link
Contributor

@Spoffy Spoffy commented Sep 21, 2024

The calendar widget currently doesn't work in Grist Desktop, due to it trying to fetch certain dependencies from the internet.

This PR adds those dependencies to the bundle, enabling offline use.

Testing

This change was tested locally by producing a new bundle and rebuilding Grist Desktop.

Copy link

netlify bot commented Sep 21, 2024

Deploy Preview for boisterous-sunburst-a5c941 ready!

Name Link
🔨 Latest commit 29dc782
🔍 Latest deploy log https://app.netlify.com/sites/boisterous-sunburst-a5c941/deploys/66ee551deb47990008f3b8bf
😎 Deploy Preview https://deploy-preview-139--boisterous-sunburst-a5c941.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.

@berhalak
Copy link
Contributor

Do you know why this happened? It used to work offline.

Copy link
Contributor

@berhalak berhalak left a comment

Choose a reason for hiding this comment

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

Ahh, we added translations. Can you add an issue (or a task) to add some kind of safe check if all external links were resolved?

@berhalak berhalak merged commit 2c440a2 into master Sep 21, 2024
1 of 2 checks passed
@jordigh jordigh deleted the spoffy/fix-calendar-bundling branch September 21, 2024 22:47
@Spoffy
Copy link
Contributor Author

Spoffy commented Sep 23, 2024

Added #140 on the issue tracker to cover this.

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.

2 participants