Skip to content

#975 | Implement Client-Side Export Range Validations#1001

Merged
pmjanus merged 2 commits intodevfrom
975-add-rages-check-on-export-page
Mar 13, 2025
Merged

#975 | Implement Client-Side Export Range Validations#1001
pmjanus merged 2 commits intodevfrom
975-add-rages-check-on-export-page

Conversation

@Viterbo
Copy link
Contributor

@Viterbo Viterbo commented Mar 5, 2025

Fixes: #975

Description

This PR implements client-side validations for export range inputs to mirror server constraints. It validates both date and block ranges to ensure they do not exceed the maximum allowed interval (one year for dates and the equivalent in blocks) and displays error messages when the inputs are invalid, while also disabling the download button to prevent 500 errors.

@Viterbo Viterbo self-assigned this Mar 5, 2025
@Viterbo Viterbo requested a review from pmjanus March 5, 2025 18:45
@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for dev-mainnet-teloscan ready!

Name Link
🔨 Latest commit b916c42
🔍 Latest deploy log https://app.netlify.com/sites/dev-mainnet-teloscan/deploys/67cf0563741a080008652a47
😎 Deploy Preview https://deploy-preview-1001--dev-mainnet-teloscan.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.

@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for testnet-teloscan ready!

Name Link
🔨 Latest commit b916c42
🔍 Latest deploy log https://app.netlify.com/sites/testnet-teloscan/deploys/67cf05636a47c10008ab25ec
😎 Deploy Preview https://deploy-preview-1001--testnet-teloscan.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.

@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for teloscan-stage ready!

Name Link
🔨 Latest commit b916c42
🔍 Latest deploy log https://app.netlify.com/sites/teloscan-stage/deploys/67cf0563167db60008324f8f
😎 Deploy Preview https://deploy-preview-1001--teloscan-stage.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.

@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for zkevm-testnet ready!

Name Link
🔨 Latest commit b916c42
🔍 Latest deploy log https://app.netlify.com/sites/zkevm-testnet/deploys/67cf0563e9cb9d000872cb7c
😎 Deploy Preview https://deploy-preview-1001--zkevm-testnet.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.

pmjanus
pmjanus previously approved these changes Mar 10, 2025
@pmjanus pmjanus merged commit 9489602 into dev Mar 13, 2025
17 checks passed
@pmjanus pmjanus deleted the 975-add-rages-check-on-export-page branch March 13, 2025 13:15
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

Comments