Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deps): update dependency graphql-scalars to v1.20.4 #7805

Merged
merged 1 commit into from
Mar 13, 2023

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 13, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
graphql-scalars 1.20.1 -> 1.20.4 age adoption passing confidence

Release Notes

Urigo/graphql-scalars

v1.20.4

Compare Source

v1.20.3

Compare Source

v1.20.2

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the release:chore This PR is a chore (means nothing for users) label Mar 13, 2023
@renovate renovate bot force-pushed the renovate/graphql-scalars-1.x branch from 40d3622 to e4f1195 Compare March 13, 2023 17:01
@replay-io
Copy link

replay-io bot commented Mar 13, 2023

16 replays were recorded for e4f1195.

image 0 Failed
image 16 Passed
    requireAuth graphql checks
          ```
          locator.waitFor: Target closed
          =========================== logs ===========================
          waiting for locator('.rw-form-error-title').locator('text=You don\'t have permission to do that') to be visible
          ============================================================
          ```
        </ol>
      </details>
      <li><a href=https://app.replay.io/recording/218511c2-0b59-4e5c-a379-df9b78d8c994>useAuth hook, auth redirects checks</a></li>
      <li><a href=https://app.replay.io/recording/c610b5c1-f827-414a-a9cc-a19c5617cfec>Check that a specific blog post is prerendered</a></li>
      <li><a href=https://app.replay.io/recording/a6fec80c-56b8-491c-ae15-b25649794318>Check that about is prerendered</a></li>
      <li><a href=https://app.replay.io/recording/4fec9bac-d1ee-4e9a-9b4d-3ae82baff3d5>Check that homepage is prerendered</a></li>
      <li><a href=https://app.replay.io/recording/3b0278fd-07d5-456d-8d37-aefc9be32ed7>Check that meta-tags are rendering the correct dynamic data</a></li>
      <li><a href=https://app.replay.io/recording/952d15ed-aaa8-40d7-8061-d18c92487ca6>Check that you can navigate from home page to specific blog post</a></li>
      <li><a href=https://app.replay.io/recording/075198d0-98b0-4919-92a4-0c0a4df5dee3>Waterfall prerendering (nested cells)</a></li>
      <li><a href=https://app.replay.io/recording/14814667-19bc-4778-af31-15729f358c49>RBAC: Admin user should be able to delete contacts</a></li>
      <li><a href=https://app.replay.io/recording/2e3783db-14f8-4365-9235-b59f91fadf56>RBAC: Should not be able to delete contact as non-admin user</a></li>
      <li><a href=https://app.replay.io/recording/762061c5-9a7c-4add-affd-a8b5f4752e61>Smoke test with dev server</a></li>
      <li><a href=https://app.replay.io/recording/487328c9-bb04-4a3c-a6d1-217ba86692ad>Smoke test with rw serve</a></li>
      <li><a href=https://app.replay.io/recording/fbeee394-a4df-4aab-bf07-73f6d54f687b>Loads Cell mocks when Cell is nested in another story</a></li>
      <li><a href=https://app.replay.io/recording/57d3e8f8-2a1a-4bec-bbf9-043549b12ca8>Loads Cell Stories</a></li>
      <li><a href=https://app.replay.io/recording/a1f9b744-639a-4fd6-bbb8-f9d2956d5923>Loads MDX Stories</a></li>
      <li><a href=https://app.replay.io/recording/d65e0242-f8fe-47ca-b88c-4fde6c27f047>Mocks current user, and updates UI while dev server is running</a></li>
      

View test run on Replay ↗︎

@jtoar jtoar merged commit 01d25af into main Mar 13, 2023
@jtoar jtoar deleted the renovate/graphql-scalars-1.x branch March 13, 2023 23:43
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Mar 13, 2023
@thedavidprice thedavidprice modified the milestones: next-release, v4.4.0 Mar 23, 2023
@jtoar jtoar added release:breaking This PR is a breaking change and removed release:chore This PR is a chore (means nothing for users) labels Mar 24, 2023
@jtoar jtoar modified the milestones: v4.4.0, v5.0.0 Mar 24, 2023
@jtoar
Copy link
Contributor

jtoar commented Mar 24, 2023

After merging this, BigInts no longer "just work". Instead you get the error:

{
  "errors": [
    {
      "message": "Something went wrong.",
      "extensions": {
        "originalError": {
          "message": "Do not know how to serialize a BigInt",
          "stack": "TypeError: Do not know how to serialize a BigInt\n    at JSON.stringify (<anonymous>)\n    at jsonStringifyResultWithoutInternals (/Users/dom/prjcts/redwood/redwood-app/node_modules/graphql-yoga/cjs/plugins/resultProcessor/stringify.js:7:17)\n    at processRegularResult (/Users/dom/prjcts/redwood/redwood-app/node_modules/graphql-yoga/cjs/plugins/resultProcessor/regular.js:21:81)\n    at processResult (/Users/dom/prjcts/redwood/redwood-app/node_modules/graphql-yoga/cjs/process-request.js:34:12)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)"
        }
      }
    }
  ]
}

Relevant commit: Urigo/graphql-scalars@15017fd

jtoar added a commit that referenced this pull request Mar 24, 2023
jtoar added a commit that referenced this pull request Mar 24, 2023
@jtoar jtoar modified the milestones: v5.0.0, chore Mar 27, 2023
@jtoar jtoar added release:chore This PR is a chore (means nothing for users) and removed release:breaking This PR is a breaking change labels Mar 27, 2023
@Josh-Walker-GM Josh-Walker-GM modified the milestones: chore, v8.0.0 Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants