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

Improve validation error messages (including oneOf) #16

Open
Bjwebb opened this issue Feb 4, 2019 · 7 comments
Open

Improve validation error messages (including oneOf) #16

Bjwebb opened this issue Feb 4, 2019 · 7 comments

Comments

@Bjwebb
Copy link
Collaborator

Bjwebb commented Feb 4, 2019

No description provided.

@odscjames
Copy link
Collaborator

This might be a dupe of a ticket in the BODS CRM?

@Bjwebb
Copy link
Collaborator Author

Bjwebb commented Feb 5, 2019

There's definitely an internal plan.io issue https://opendataservices.plan.io/issues/19384.
I deliberately created an extra public issue, to link my public commits against. Does this make sense?

@Bjwebb
Copy link
Collaborator Author

Bjwebb commented Feb 5, 2019

Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 5, 2019
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Feb 5, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 5, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 5, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 5, 2019
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Feb 5, 2019
@kd-ods
Copy link
Collaborator

kd-ods commented Feb 5, 2019

[Related issue now on the BODS agile board: http://bods.opendataservices.coop/redmine/issues/436]

@kd-ods
Copy link
Collaborator

kd-ods commented Feb 5, 2019

Minor text improvement suggestion: instead of "'xxxx' is too short", have "'statementID 'xxxx' is too short"

(If we try to get a field name into all Error Descriptions, users can glance down the Error Description column and see in a one-er which fields are causing problems.)

Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Feb 7, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 18, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 18, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Feb 18, 2019
@Bjwebb Bjwebb changed the title Improve validation error messages for oneOf Improve validation error messages (including oneOf) Feb 18, 2019
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Feb 19, 2019
@Bjwebb
Copy link
Collaborator Author

Bjwebb commented Feb 19, 2019

Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Feb 25, 2019
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Mar 22, 2019
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Mar 25, 2019
…ges""

This reverts commit 9f654d0,
effectively reapplying commit cf824e8.
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Mar 28, 2019
This will allow individual project libraries (e.g. lib-cove-bods) to
rewrite their own error messages more easily.
Bjwebb added a commit to OpenDataServices/lib-cove that referenced this issue Mar 28, 2019
Bjwebb added a commit to openownership/lib-cove-bods that referenced this issue Mar 28, 2019
@Bjwebb
Copy link
Collaborator Author

Bjwebb commented Apr 1, 2019

I've refactored the work that links to this issues.

I've split out the oneOf work, that has been merged in these two PRs:

I've moved the validation message changes into lib-cove-bods, although that requires some changes to lib-cove to put more information into the error JSON.

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

No branches or pull requests

3 participants