Apply suggestions from code review

Co-authored-by: Kasper Overgård Nielsen <[email protected]>
realm · Jun 14, 2024 · 3e69da1 · 3e69da1
1 parent ad7f178
commit 3e69da1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -52,7 +52,7 @@
 * Report the originating error that caused a client reset to occur. (Core 14.9.0)
 * Allow the realm package, and code generated by realm_generator to be included when building 
   for web without breaking compilation. (Issue [#1374](https://github.com/realm/realm-dart/issues/1374))
-* Added support for specifying key paths when listening to notifications on a collection with the `changesFor([List<String>? keyPaths])` method, available on `RealmResults`, `RealmList`, `RealmSet`, `RealmMap`. The key paths indicates which changes in properties should raise a notification.
+* Added support for specifying key paths when listening to notifications on a collection with the `changesFor([List<String>? keyPaths])` method. Available on `RealmResults`, `RealmList`, `RealmSet`, and `RealmMap`. The key paths indicates what properties should raise a notification, if changed either directly or transitively.
   ```dart
   @RealmModel()
   class _Person {

diff --git a/packages/realm_dart/lib/src/handles/native/object_handle.dart b/packages/realm_dart/lib/src/handles/native/object_handle.dart
@@ -136,7 +136,7 @@ class ObjectHandle extends RootedHandleBase<realm_object> implements intf.Object
   }
 
   @override
-  NotificationTokenHandle subscribeForNotifications(NotificationsController controller, [List<String>? keyPaths]) {
+  NotificationTokenHandle subscribeForNotifications(NotificationsController controller, List<String>? keyPaths, int? classKey) {
     return using((arena) {
       final kpNative = root.buildAndVerifyKeyPath(keyPaths, classKey);
       return NotificationTokenHandle(