Merge pull request #324 from cofacts/nullable-createdat

Nullable createdat
cofacts · Oct 26, 2023 · 4b00d58 · 4b00d58
2 parents 3a215c4 + 5636d7a
commit 4b00d58
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/src/graphql/models/Article.js b/src/graphql/models/Article.js
@@ -61,7 +61,10 @@ const Article = new GraphQLObjectType({
   fields: () => ({
     id: { type: new GraphQLNonNull(GraphQLID) },
     text: { type: GraphQLString },
-    createdAt: { type: new GraphQLNonNull(GraphQLString) },
+    createdAt: {
+      type: GraphQLString,
+      description: 'May be null for legacy articles',
+    },
     updatedAt: { type: GraphQLString },
     status: { type: new GraphQLNonNull(ReplyRequestStatusEnum) },
     references: { type: new GraphQLList(ArticleReference) },

diff --git a/src/graphql/models/ArticleReply.js b/src/graphql/models/ArticleReply.js
@@ -134,7 +134,10 @@ export default new GraphQLObjectType({
       resolve: ({ status }) => (status === undefined ? 'NORMAL' : status),
     },
 
-    createdAt: { type: new GraphQLNonNull(GraphQLString) },
+    createdAt: {
+      type: GraphQLString,
+      description: 'May be null for legacy article-replies',
+    },
     updatedAt: { type: GraphQLString },
   }),
 });
diff --git a/src/graphql/models/Reply.js b/src/graphql/models/Reply.js
@@ -31,7 +31,10 @@ const Reply = new GraphQLObjectType({
       description: 'The user submitted this reply version',
       resolve: userFieldResolver,
     },
-    createdAt: { type: new GraphQLNonNull(GraphQLString) },
+    createdAt: {
+      type: GraphQLString,
+      description: 'May be null for legacy replies',
+    },
     text: { type: GraphQLString },
     type: { type: new GraphQLNonNull(ReplyTypeEnum) },
     reference: { type: GraphQLString },