Merge pull request #228 from MacondoExpress/deprecates-id-aggregations

Add deprecations for ID aggregations.
neo4j · Jan 13, 2025 · f4f3a97 · f4f3a97
2 parents 08e9c96 + 7dbc230
commit f4f3a97
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/modules/ROOT/pages/migration/index.adoc b/modules/ROOT/pages/migration/index.adoc
@@ -560,3 +560,7 @@ A warning is raised if you use a deprecated value.
 === Deprecated `typename_IN` filter in favor of `typename`
 
 `typename_IN` filter has been deprecated in favor of `typename`.
+
+=== Deprecated aggregations on ID fields
+
+Aggregations on ID fields are now deprecated and will be removed in the future.
diff --git a/modules/ROOT/pages/optimization.adoc b/modules/ROOT/pages/optimization.adoc
@@ -29,6 +29,7 @@ const neoSchema = new Neo4jGraphQL({
             directedArgument: true,
             connectOrCreate: true,
             typename_IN: true,
+            idAggregations: true,
         },
     },
 });

diff --git a/modules/ROOT/pages/queries-aggregations/aggregations.adoc b/modules/ROOT/pages/queries-aggregations/aggregations.adoc
@@ -49,7 +49,7 @@ Based on the type definitions, here is a list of fields that accept aggregations
 |===
 | Type | Aggregating function | Example
 
-| String (e.g. `ID`, `String`)
+| String (e.g. `String`)
 | `shortest`, `longest`
 a| 
 .Longest User name