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

Enhance error handling and add checks for missing files or inconsiste… #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KRISHNPRIY2820
Copy link

@KRISHNPRIY2820 KRISHNPRIY2820 commented Feb 15, 2025

Enhanced Error Handling and Robustness for Order.json

Summary

This pull request improves error handling and adds checks for robustness in handling order.json and Markdown files. The changes ensure that users receive informative error messages when syntax errors or inconsistencies occur.

Changes Made

  1. Enhanced MDsveX Error Handling:

    • Added configuration to print warnings and offending files in the browser console.
    • Improved user experience by providing clearer error messages.
  2. Robust Handling of order.json:

    • Added checks to ensure consistency between order.json
    • Printed informative error messages when discrepancies are found.

Impact

  • Improved usability for non-developers by providing better error messages.
  • Increased robustness in handling order.json

Verification

  • Tested the changes locally to ensure error messages are printed correctly.
  • Verified that the system handles missing files and inconsistencies gracefully.

Please review and provide feedback.

Below is the modified code for
svelte.config.js
image

vite.config.js
image

created a new file order.js

image

Issue #12 better errors for usability #12 #12

Copy link

netlify bot commented Feb 15, 2025

Deploy Preview for iil-is failed.

Name Link
🔨 Latest commit a64b617
🔍 Latest deploy log https://app.netlify.com/sites/iil-is/deploys/67b05170b0b85800084776c1

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.

1 participant