Skip to content

fix: missing FormQueryParamsProvider in account view#5270

Merged
jacobsfletch merged 1 commit into
payloadcms:alphafrom
willviles:fix/missing-form-query-params-provider
Mar 8, 2024
Merged

fix: missing FormQueryParamsProvider in account view#5270
jacobsfletch merged 1 commit into
payloadcms:alphafrom
willviles:fix/missing-form-query-params-provider

Conversation

@willviles
Copy link
Copy Markdown
Contributor

Description

Updating the current user's account on /admin/account causes the following error:

Unhandled Runtime Error
TypeError: dispatchFormQueryParams is not a function

Call Stack
dispatchFormQueryParams
node_modules/.pnpm/@payloadcms+next@3.0.0-alpha.34_@types+react@18.2.64_http-status@1.6.2_monaco-editor@0.46.0_n_n7tiwpnlqwhrajzmafbvh4ooai/node_modules/@payloadcms/next/dist/views/Edit/index.client.js (22:13)
onSaveFromContext
node_modules/.pnpm/@payloadcms+next@3.0.0-alpha.34_@types+react@18.2.64_http-status@1.6.2_monaco-editor@0.46.0_n_n7tiwpnlqwhrajzmafbvh4ooai/node_modules/@payloadcms/next/dist/views/Edit/Default/index.js (50:18)
onSuccess
node_modules/.pnpm/@payloadcms+ui@3.0.0-alpha.34_@types+react@18.2.64_monaco-editor@0.46.0_next@14.2.0-canary.7__tu76glckrdk3y2gcvr5mf4lzvq/node_modules/@payloadcms/ui/dist/forms/Form/index.js (181:54)

This PR wraps the Account view with the appropriate FormQueryParamsProvider to provide the view with the dispatchFormQueryParams function.

  • I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have added tests that prove my fix is effective or that my feature works
  • Existing test suite passes locally with my changes
  • I have made corresponding changes to the documentation

@jacobsfletch jacobsfletch merged commit 45a4439 into payloadcms:alpha Mar 8, 2024
@jacobsfletch
Copy link
Copy Markdown
Member

THANK YOU @willviles 🙏 👏 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants