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.

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

fix: adjust data deserializer to return correct type #12

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

jboix
Copy link
Contributor

@jboix jboix commented Nov 5, 2024

Description

Fixes a bug in EventService where deserialization of the data field was failing.

Changes Made

  • Changed the DataDeserializer to return Any? instead of JsonNode to fix type mismatch errors in 'data'field deserialization.
  • Changed the user agent field from agent to user_agent to match specificiations.

Checklist

  • I have followed the project's style and contribution guidelines.
  • I have performed a self-review of my own changes.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.

- Changed the DataDeserializer to return Any? instead of JsonNode to fix type mismatch errors
  in 'data'field deserialization.
- Changed the user agent field from `agent` to `user_agent` to match specificiations.
@jboix jboix requested a review from MGaetan89 November 5, 2024 10:26
@jboix jboix self-assigned this Nov 5, 2024
@jboix jboix added this pull request to the merge queue Nov 5, 2024
Merged via the queue into main with commit 8bc123f Nov 5, 2024
1 check passed
@jboix jboix deleted the fix/data-deserializer-return-type branch November 5, 2024 13:07
Copy link

github-actions bot commented Nov 5, 2024

🎉 This PR is included in version 2.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants