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.

Sign up for GitHub

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

Jump to bottom

set contentType when invoke binding #1249

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

set contentType when invoke binding #1249

wants to merge 7 commits into from
+8 −5

Conversation

theonefx
Copy link
Contributor

@theonefx theonefx commented Mar 11, 2025 •
edited
Loading

Description

  • Use object serializer contentType when invoke binding.
  • Optimize set contentType when saveSate, make sure saveBulkState & saveState using same logic.

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@theonefx theonefx requested review from a team as code owners March 11, 2025 10:22
@theonefx theonefx force-pushed the add_content_type_header branch from 720195e to 42f8738 Compare March 11, 2025 10:30
@theonefx
Copy link
Contributor Author

@artursouza

@theonefx theonefx changed the title set content-type when invoke binding set contentType when invoke binding Mar 12, 2025
@theonefx
code style check done
88479d3 
Signed-off-by: theonefx <[email protected]>
@theonefx theonefx force-pushed the add_content_type_header branch from b0bfe36 to 88479d3 Compare March 12, 2025 02:33
theonefx added 2 commits March 12, 2025 10:39
@theonefx
bugfix
4d87ef0 
Signed-off-by: theonefx <[email protected]>
@theonefx
retest
753f0ee 
Signed-off-by: theonefx <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@theonefx