Skip to content

fix(urlParams): empty save param issue#549

Merged
floryst merged 1 commit intoKitware:mainfrom
jadh4v:fix-empty-urlParams-error
Feb 8, 2024
Merged

fix(urlParams): empty save param issue#549
floryst merged 1 commit intoKitware:mainfrom
jadh4v:fix-empty-urlParams-error

Conversation

@jadh4v
Copy link
Collaborator

@jadh4v jadh4v commented Feb 8, 2024

Fix the following error when urlParams are empty.

App.vue:206 Uncaught TypeError: Cannot read properties of undefined (reading 'toString')
    at setup (App.vue:206:54)
    at callWithErrorHandling (chunk-G4DFXOZZ.js?v=fae193fd:1565:18)
    at setupStatefulComponent (chunk-G4DFXOZZ.js?v=fae193fd:8631:25)
    at setupComponent (chunk-G4DFXOZZ.js?v=fae193fd:8592:36)
    at mountComponent (chunk-G4DFXOZZ.js?v=fae193fd:6997:7)
    at processComponent (chunk-G4DFXOZZ.js?v=fae193fd:6963:9)
    at patch (chunk-G4DFXOZZ.js?v=fae193fd:6436:11)
    at render2 (chunk-G4DFXOZZ.js?v=fae193fd:7730:7)
    at mount (chunk-G4DFXOZZ.js?v=fae193fd:5215:13)
    at app.mount (chunk-G4DFXOZZ.js?v=fae193fd:10537:19)

@netlify
Copy link

netlify bot commented Feb 8, 2024

Deploy Preview for volview-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 7afb866
🔍 Latest deploy log https://app.netlify.com/sites/volview-dev/deploys/65c44778450879000870af53
😎 Deploy Preview https://deploy-preview-549--volview-dev.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.

@jadh4v jadh4v requested a review from floryst February 8, 2024 03:17
@floryst floryst added this pull request to the merge queue Feb 8, 2024
Merged via the queue into Kitware:main with commit cb2e832 Feb 8, 2024
PaulHax pushed a commit to PaulHax/VolView that referenced this pull request Apr 24, 2025
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