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

feat: add updated error message to invalid data #200

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

rozanagy
Copy link
Contributor

This pull request includes a refactor to the error message handling in the LedgerModalErrorBox component to improve maintainability and readability.

Error message handling improvements:

Refactored the formatErrorMessage function to use a Record object for error messages, simplifying the code and making it easier to add new error messages in the future.

@rozanagy rozanagy requested a review from Polybius93 October 23, 2024 09:12
Copy link

netlify bot commented Oct 23, 2024

Deploy Preview for test-mint-dlc-link ready!

Name Link
🔨 Latest commit 36be173
🔍 Latest deploy log https://app.netlify.com/sites/test-mint-dlc-link/deploys/6718be13ef6f280009e9d777
😎 Deploy Preview https://deploy-preview-200--test-mint-dlc-link.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.

Copy link

netlify bot commented Oct 23, 2024

Deploy Preview for dev-mint-dlc-link ready!

Name Link
🔨 Latest commit 36be173
🔍 Latest deploy log https://app.netlify.com/sites/dev-mint-dlc-link/deploys/6718be13ea580e00082e8cb5
😎 Deploy Preview https://deploy-preview-200--dev-mint-dlc-link.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.

@Polybius93 Polybius93 merged commit 2d0c747 into dev Oct 23, 2024
15 checks passed
@Polybius93 Polybius93 deleted the feat/format-error-message-for-ledger-error branch October 23, 2024 13:50
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