-
Notifications
You must be signed in to change notification settings - Fork 87
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
binary type support #1320
binary type support #1320
Conversation
bump agp version
anonymize bundle id
# Conflicts: # lib/src/native/realm_core.dart
@@ -16,7 +16,8 @@ | |||
## 1.2.0 (2023-06-08) | |||
|
|||
### Enhancements | |||
* Added support for Full-Text search (simple term) queries. ([#1300](https://github.com/realm/realm-dart/pull/1300)) | |||
* Added support binary data type. ([#1320](https://github.com/realm/realm-dart/pull/1320)) | |||
* Added support for Full-Text search (simple term) queries. ([#1300](https://github.com/realm/realm-dart/pull/1300)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was just formatted
Pull Request Test Coverage Report for Build 5311844051
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already discussed on round-table yesterday. To summarize:
- I like that we use
Uint8LIst
on the interface. - I don't off the top of my mind have a better way to handle default values for binary.
- We could consider creating a
Matcher
that knows how to compareUint8List
s to avoid special cases in the tests, but I don't think it important. - Hopefully we can improve performance once we use the new binding generator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me.
Co-authored-by: Desislava Stefanova <95419820+desistefanova@users.noreply.github.com>
Adds Binary type support
fixes #1185