Change 'id' argument's type to ID in page query

torchbox · Aug 2, 2023 · 9743490 · 9743490
1 parent 1bd7972
commit 9743490
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/docs/general-usage/graphql-types.rst b/docs/general-usage/graphql-types.rst
@@ -80,7 +80,7 @@ The singular ``page`` field accepts the following arguments:
 
 ::
 
-    id: Int                       # Can be used on it's own
+    id: ID                        # Can be used on it's own
     slug: String                  # Can be used on it's own
     urlPath: String               # Can be used on it's own
     token: String                 # Must be used with one of the others. Usually contentType

diff --git a/example/example/tests/test_grapple.py b/example/example/tests/test_grapple.py
@@ -327,7 +327,7 @@ def test_pages_content_type_filter(self):
 
     def test_page(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 contentType
                 parent {

diff --git a/example/home/test/test_blog.py b/example/home/test/test_blog.py
@@ -131,7 +131,7 @@ def setUp(self):
 
     def test_blog_page(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     title
@@ -146,7 +146,7 @@ def test_blog_page(self):
 
     def test_related_author_page(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     author {
@@ -166,7 +166,7 @@ def test_related_author_page(self):
 
     def get_blocks_from_body(self, block_type, block_query="rawValue", page_id=None):
         query = """
-        query($id: Int) {{
+        query($id: ID) {{
             page(id: $id) {{
                 ... on BlogPage {{
                     body {{
@@ -246,7 +246,7 @@ def test_streamfield_richtextblock(self):
 
     def test_richtext(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     summary
@@ -468,7 +468,7 @@ def test_blog_body_objectives(self):
 
     def test_blog_embed(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     body {
@@ -650,7 +650,7 @@ def test_blog_body_snippetchooserblock_person(self):
     # Next 2 tests are used to test the Collection API, both ForeignKey and nested field extraction.
     def test_blog_page_related_links(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     relatedLinks {
@@ -670,7 +670,7 @@ def test_blog_page_related_links(self):
 
     def test_blog_page_related_urls(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     relatedUrls
@@ -690,7 +690,7 @@ def test_blog_page_paginated_authors(self):
         per_page = 5
 
         query = """
-        query ($id: Int, $page: PositiveInt, $perPage: PositiveInt) {
+        query ($id: ID, $page: PositiveInt, $perPage: PositiveInt) {
             page(id: $id) {
                 ... on BlogPage {
                     paginatedAuthors(page: $page, perPage: $perPage) {
@@ -899,7 +899,7 @@ def test_singular_blog_page_query(self):
 
     def test_blog_page_tags(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     tags {
@@ -1037,7 +1037,7 @@ def test_graphqlfield_method_in_structblock(self):
 
     def test_custom_property(self):
         query = """
-        query($id: Int) {
+        query($id: ID) {
             page(id: $id) {
                 ... on BlogPage {
                     customProperty

diff --git a/grapple/types/pages.py b/grapple/types/pages.py
@@ -336,7 +336,7 @@ class Mixin:
         )
         page = graphene.Field(
             PageInterface,
-            id=graphene.Int(),
+            id=graphene.ID(),
             slug=graphene.String(),
             url_path=graphene.Argument(
                 graphene.String,