Fix User Parameter Validation and SVG Error Handling - #9023
Fix User Parameter Validation and SVG Error Handling#9023armanpanigrahi59 wants to merge 4 commits into
Conversation
|
@armanpanigrahi59 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
This PR improves user parameter validation and error handling for SVGs, which is crucial for maintaining a smooth user experience. It aligns with previous decisions to enhance visual elements and user interactions, such as the addition of a dark/light preview background toggle and language-aware visualizations. Thank you for your contributions, and I look forward to your updates! |
📦 Next.js Bundle Size Report (Gzipped Sizes)
📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
The error handling and validation fix for JSON responses is good, but your branch is polluted with unrelated commits (like the Docker builds, timezone mocks, and webhook handler changes). Could you please rebase your branch or isolate just the SVG/JSON error handling changes into a clean branch? We can't merge this while it contains unrelated commits from other PRs.
Description
Fixes #870
Pillar
Visual Preview
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.