Skip to content

Support readOnly property from latest validation draft#1282

Merged
epicfaace merged 4 commits intorjsf-team:masterfrom
pahen:readOnly
May 14, 2019
Merged

Support readOnly property from latest validation draft#1282
epicfaace merged 4 commits intorjsf-team:masterfrom
pahen:readOnly

Conversation

@pahen
Copy link
Contributor

@pahen pahen commented May 13, 2019

Reasons for making this change

Since the readOnly property now is available in the latest validation draft (previously it was in hyper schema only), it makes sense to use this property to signal read only for fields.

Fixes #976
Replaces PR #888

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests
    • I've updated docs if needed
    • I've run npm run cs-format on my branch to conform my code to prettier coding style
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

@pahen pahen changed the title Read only Support readOnly property from latest validation draft May 13, 2019
Copy link
Member

@epicfaace epicfaace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@epicfaace epicfaace merged commit 98952b7 into rjsf-team:master May 14, 2019
@pahen
Copy link
Contributor Author

pahen commented May 14, 2019

Would it be possible to have a new release with this included soon?

@pahen
Copy link
Contributor Author

pahen commented Jun 5, 2019

Is there any plan for a new release soon @epicfaace? I really need this feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't work "readOnly" attribute from Schema

3 participants