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 input value for Decimal validate hook #9262

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

kennedybaird
Copy link
Contributor

Solve for #8597 (followed @borisno2's recommendation)

@kennedybaird kennedybaird changed the title fix: cast field input to Decimal fix: cast field input to Decimal in validate hooks Aug 6, 2024
@kennedybaird kennedybaird force-pushed the fix/decimal-validation branch from c5d4ce8 to f2b1187 Compare August 20, 2024 20:58
Copy link

codesandbox-ci bot commented Aug 20, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit f8afad4:

Sandbox Source
@keystone-6/sandbox Configuration

@kennedybaird
Copy link
Contributor Author

Added the changeset.

@acburdine - maybe if you have a minute you could review this to save Daniel some time?

@dcousens dcousens merged commit d543e60 into keystonejs:main Sep 3, 2024
43 of 44 checks passed
@dcousens dcousens changed the title fix: cast field input to Decimal in validate hooks Fix input value for Decimal validate hook Sep 3, 2024
relusion added a commit to relusion/keystone that referenced this pull request Sep 8, 2024
- Remove unused code and imports in backend files
- Add bootstrap v5.3.3 as a frontend dependency
- Update package-lock.json and package.json
- Update App.js to import bootstrap CSS
- Improve loading state in App.js
- Add error handling and fetch posts functionality in App.js
- Add addNewPost functionality in App.js
- Update UI in App.js to display user information and posts

Related to keystonejs#9322, keystonejs#9320, keystonejs#9262
@dcousens dcousens mentioned this pull request Nov 5, 2024
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