Skip to content

Use UUIDEncoder to dump message with UUID#685

Merged
kumaranvpl merged 1 commit into
mainfrom
fix-error
Apr 15, 2025
Merged

Use UUIDEncoder to dump message with UUID#685
kumaranvpl merged 1 commit into
mainfrom
fix-error

Conversation

@kumaranvpl

@kumaranvpl kumaranvpl commented Apr 15, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

We already use UUIDEncoder for json.dumps when a event includes UUID. This place was missed and I fixed it.

Type of change

  • Documentation (typos, code examples, or any documentation updates)
  • Bug fix (a non-breaking change that resolves an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would disrupt existing functionality)
  • This change requires a documentation update

Checklist

  • I have conducted a self-review of my own code
  • I have made the necessary changes to the documentation
  • My changes do not generate any new warnings
  • I have added tests to validate the effectiveness of my fix or the functionality of my new feature
  • I have included code examples to illustrate the modifications

@kumaranvpl kumaranvpl requested a review from rjambrecic April 15, 2025 08:31
@kumaranvpl kumaranvpl added this pull request to the merge queue Apr 15, 2025
Merged via the queue into main with commit 0d1af1e Apr 15, 2025
@kumaranvpl kumaranvpl deleted the fix-error branch April 15, 2025 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@rjambrecic rjambrecic rjambrecic approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@rjambrecic rjambrecic

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kumaranvpl @rjambrecic