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

Have more JSON serialization controls on JsonCommands Set methods. #222

Closed
RickyLin opened this issue Dec 8, 2023 · 0 comments · Fixed by #223
Closed

Have more JSON serialization controls on JsonCommands Set methods. #222

RickyLin opened this issue Dec 8, 2023 · 0 comments · Fixed by #223

Comments

@RickyLin
Copy link
Contributor

RickyLin commented Dec 8, 2023 •
edited
Loading

NRedisStack Version: 0.10.1

Redis Stack Version: 7.2.3

Description:
We may need more JSON serialization controls on how object is serialized. It should be easy to add a JsonSerializerOptions parameter to Set, SetAsync, Merge and MergeAsync method of JsonCommands type.
RickyLin added a commit to RickyLin/NRedisStack that referenced this issue Dec 8, 2023
@RickyLin
Added JsonSerializerOptions parameter to Set and SetAsync methods.
7a648d6 
Closed redis#222
@RickyLin RickyLin mentioned this issue Dec 8, 2023
Merged
RickyLin added a commit to RickyLin/NRedisStack that referenced this issue Dec 8, 2023
@RickyLin
Added JsonSerializerOptions parameter to Set and SetAsync methods.
9f2b62b 
Closed redis#222
RickyLin added a commit to RickyLin/NRedisStack that referenced this issue Dec 8, 2023
@RickyLin
Added JsonSerializerOptions parameter to Set and SetAsync methods.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
c1a91b1 
Closed redis#222
shacharPash added a commit that referenced this issue Dec 11, 2023
@RickyLin @shacharPash
Added JsonSerializerOptions parameter to Set and SetAsync methods. (#223
f6a251f 
)

* Added JsonSerializerOptions parameter to Set and SetAsync methods.
Closed #222

* format

* { get; set; } for Birthday

* remove  { get; set; }

---------

Co-authored-by: shacharPash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added JsonSerializerOptions parameter to Set and SetAsync methods. RickyLin/NRedisStack
1 participant
@RickyLin