Skip to content

Comments

[ui-core] auto set qsEncodeData key in useSaveData hook#4082

Merged
ramprasadagarwal merged 6 commits intomasterfrom
improv/useSaveData-hook-qsEndode
Apr 3, 2025
Merged

[ui-core] auto set qsEncodeData key in useSaveData hook#4082
ramprasadagarwal merged 6 commits intomasterfrom
improv/useSaveData-hook-qsEndode

Conversation

@ramprasadagarwal
Copy link
Collaborator

What changes were proposed in this pull request?

  • Reduces the repetition of passing qsEncodeData: false key in useSaveData whenever the payload is of FormData.

How was this patch tested?

  • Added Unit test
  • Manually tested the APIs

Please review Hue Contributing Guide before opening a pull request.

@github-actions
Copy link

github-actions bot commented Mar 26, 2025

✅ Test files were modified. Ensure that the tests cover all relevant changes

@github-actions
Copy link

github-actions bot commented Mar 26, 2025

Backend Codecov

Backend Code Coverage Report •
FileStmtsMissCoverMissing
TOTAL535802625151% 
report-only-changed-files is enabled. No files were changed during this commit :)

Pytest Report

Tests Skipped Failures Errors Time
1101 106 💤 0 ❌ 0 🔥 5m 56s ⏱️

@ramprasadagarwal ramprasadagarwal enabled auto-merge (squash) April 2, 2025 15:42
@ramprasadagarwal ramprasadagarwal merged commit a0a6cd2 into master Apr 3, 2025
8 checks passed
@ramprasadagarwal ramprasadagarwal deleted the improv/useSaveData-hook-qsEndode branch April 3, 2025 07:40
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.

3 participants