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

detailed/extended error messages #238

Conversation

HonoluluHenk
Copy link
Contributor

return constraint args in ValidationError in a backwards compatible way.

Might help a lot with:

@jrjohnson
Copy link

Would be interested to know why this is lingering with no review. It seems to be a big part of what is needed to translate error messages without having to re-implement each validator. If it's a bad approach could you suggest something better and I'll take a shot at a pull request?

@NoNameProvided NoNameProvided changed the base branch from master to develop August 2, 2020 11:50
@NoNameProvided
Copy link
Member

Hi!

Sorry for the late review. Can you rebase this to the current develop (or create a new PR). You will have to

  • update your tests as we moved to Jest from Mocha
  • replace the class definition with an interface, adding it into src/interfaces

@NoNameProvided NoNameProvided added the status: awaiting answer Awaiting answer from the author of issue or PR. label Aug 4, 2020
@candunaj

This comment has been minimized.

@glen-84
Copy link

glen-84 commented Jan 24, 2021

@HonoluluHenk Will you continue working on this PR?

@jannomeister
Copy link

is this still going to be reviewed?

@HonoluluHenk
Copy link
Contributor Author

Unfortunately, I cannot invest anymore time into this PR :(

@NoNameProvided NoNameProvided removed the status: awaiting answer Awaiting answer from the author of issue or PR. label Nov 20, 2022
@NoNameProvided
Copy link
Member

NoNameProvided commented Nov 20, 2022

Closing this as the author noted he is not interested in pushing this anymore.

If someone wants to pick this up, please open a feature idea first on the discussion tab to decide how we want to approach this.

@typestack typestack locked as resolved and limited conversation to collaborators Nov 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

6 participants