fix: handle calendar fetch failure and prevent UI crash#2367
fix: handle calendar fetch failure and prevent UI crash#2367tusharv01 wants to merge 1 commit intojson-schema-org:mainfrom
Conversation
|
Hi @tusharv01! Thanks a lot for your contribution! I noticed that the following required information is missing or incomplete: issue reference Please update the PR description to include this information. You can find placeholders in the PR template for these items. Thanks a lot! |
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2367 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 30 30
Lines 672 672
Branches 211 211
=========================================
Hits 672 672 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @tusharv01! Thanks a lot for your contribution! I noticed that the following required information is missing or incomplete: issue reference Please update the PR description to include this information. You can find placeholders in the PR template for these items. Thanks a lot! |
1 similar comment
|
Hi @tusharv01! Thanks a lot for your contribution! I noticed that the following required information is missing or incomplete: issue reference Please update the PR description to include this information. You can find placeholders in the PR template for these items. Thanks a lot! |
Tushar8466
left a comment
There was a problem hiding this comment.
Nice fix! This change improves stability by handling calendar fetch failures and preventing a potential UI crash 👍
What kind of change does this PR introduce?
Bugfix
Issue Number:
Before: Homepage crashed due to undefined
datesInfoAfter: Website loads successfully with fallback handling
If relevant, did you update the documentation?
No
Summary
This PR fixes a crash on the homepage caused by failure to fetch the external Google Calendar.
This issue occurs when the external calendar API is unavailable (e.g., DNS failure, firewall blocking, or network issues). The fix ensures the app does not crash in such cases.
Previously:
datesInfobecameundefinedgetStaticProps.map()on a null/undefined valueFix:
datesInfofalls back tonullImpact:
Does this PR introduce a breaking change?
No
Checklist