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

OPHJOD-1313: Update hookform/resolvers #376

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

ketsappi
Copy link
Contributor

@ketsappi ketsappi commented Feb 25, 2025

Description

Update @hookform/resolvers and make changes to get rid of errors after update

Notes

  • Multiple places had the id as required but it type was string|undefined. Coming from the API-schema.
  • Multiple places had the loppuPvm as required when it was actually optional.
  • nimi & kuvaus were missing from multiple places.

Related JIRA ticket

https://jira.eduuni.fi/browse/OPHJOD-1313
https://jira.eduuni.fi/browse/OPHJOD-851

@ketsappi ketsappi force-pushed the OPHJOD-1313_hookform-resolvers branch from 7028021 to da6829d Compare February 25, 2025 10:59
@ketsappi ketsappi force-pushed the OPHJOD-1313_hookform-resolvers branch from da6829d to 673069d Compare February 25, 2025 11:03
@ketsappi ketsappi requested review from juhaniko and sauanto February 25, 2025 11:06
@ketsappi ketsappi force-pushed the OPHJOD-1313_hookform-resolvers branch from 673069d to c1f9b23 Compare February 25, 2025 11:08
@ketsappi ketsappi merged commit de8c02c into main Feb 25, 2025
8 checks passed
@ketsappi ketsappi deleted the OPHJOD-1313_hookform-resolvers branch February 25, 2025 12:33
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