Skip to content

3.0 rc #19

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

Open
wants to merge 104 commits into
base: master
Choose a base branch
from
Open

3.0 rc #19

wants to merge 104 commits into from

Conversation

shmax
Copy link
Owner

@shmax shmax commented Oct 18, 2024

Rewrite library with a new approach:

  • Divided old UserErrorsType class into a an abstract ErrorType base and subclasses for each of the types (ie. 'StringErrorType, InputObjectErrorType`, etc
  • Moved the validation work that was formerly happening in ValidatedFieldDefinition to the ErrorType family of classes
  • For listOf type, you can now do separate validation for the list of items as a whole, as well as each individual item in the list
  • there is no longer any concept of subErrors for input object types; to mitigate the possibility of field name conflicts, code and msg have been renamed to __code and __msg. Same goes for most of the other generated fields ('__valid', __result, __path)
  • Created own namespace for everything instead of piggy-backing on existing ones

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.

1 participant