Skip to content

fix(App): keep query params on save url query param#556

Merged
PaulHax merged 1 commit intoKitware:mainfrom
PaulHax:save-param
Mar 1, 2024
Merged

fix(App): keep query params on save url query param#556
PaulHax merged 1 commit intoKitware:mainfrom
PaulHax:save-param

Conversation

@PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Mar 1, 2024

Avoid dropping JSON or array query param arguments on the "save" query parameter by parsing query params without casting to native types in vtkURLExtract.

Now save parameter could be
/api/v1/folder/65e1f44870447d7525ac0c49/volview?metadata={"linkedResources":{"items":["65e1f4ec70447d7525ac0c54"]}}

@netlify
Copy link

netlify bot commented Mar 1, 2024

Deploy Preview for volview-dev ready!

Built without sensitive environment variables

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

@PaulHax PaulHax requested a review from floryst March 1, 2024 16:00
Avoid dropping JSON or array query param arguments on the "save" query parameter
by parsing query params without casting to native types in vtkURLExtract.
@PaulHax PaulHax added this pull request to the merge queue Mar 1, 2024
Merged via the queue into Kitware:main with commit 3cadfb0 Mar 1, 2024
@PaulHax PaulHax deleted the save-param branch March 1, 2024 16:25
PaulHax added a commit to PaulHax/VolView that referenced this pull request Apr 24, 2025
fix(App): keep query params on save url query param
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