fixup: Add TODOs for bugfixes

MystenLabs · Apr 29, 2024 · 1b8d312 · 1b8d312
1 parent 83bb808
commit 1b8d312
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/crates/sui-graphql-rpc/src/types/dynamic_field.rs b/crates/sui-graphql-rpc/src/types/dynamic_field.rs
@@ -108,6 +108,8 @@ impl DynamicField {
                 ctx,
                 self.df_object_id,
                 Object::under_parent(
+                    // TODO (RPC-131): The dynamic object field value's version should be bounded by
+                    // the field's parent version, not the version of the field object itself.
                     self.super_.super_.version_impl(),
                     self.super_.super_.checkpoint_viewed_at,
                 ),

diff --git a/crates/sui-graphql-rpc/src/types/object.rs b/crates/sui-graphql-rpc/src/types/object.rs
@@ -1337,6 +1337,7 @@ fn objects_query(filter: &ObjectFilter, range: AvailableRange, page: &Page<Curso
 where
 {
     let view = if filter.object_keys.is_some() || !filter.has_filters() {
+        // TODO (RPC-142): Fix handling of filters that contain both object keys and object ids.
         View::Historical
     } else {
         View::Consistent