Merge pull request #8 from blopez2010/fix-missing-limit-offset-error

Fixing missing limit/offset error.
blopez2010 · Oct 5, 2019 · 5901673 · 5901673
2 parents 4880f69 + 4b56858
commit 5901673
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/src/queries/people.js b/src/queries/people.js
@@ -5,10 +5,6 @@ const { paginate } = require('./extensions');
 
 module.exports = {
   allPeoplePaginated: async (parent, { name, offset, limit }) => {
-    if (!offset || !limit) {
-      throw new Error('Missing offset and/or limit fields');      
-    }
-
     const where = name ? {
       name: {
         [Op.like]: `%${name.trim()}%`

diff --git a/src/schema/schema.graphql b/src/schema/schema.graphql
@@ -4,7 +4,7 @@ type Query {
 	allEvents: [Event!]!
 	getEventByYear(year: String!): Event
 	allPeople: [People!]!
-	allPeoplePaginated(name: String, offset: Int, limit: Int): PeoplePaginationResultPayload!
+	allPeoplePaginated(name: String, offset: Int!, limit: Int!): PeoplePaginationResultPayload!
 	findPeopleByName(name: String!): [People!]!
 	allItems: [Item!]!
 	getItemsByEvent(year: Int!): [Item!]