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

editor: fix editor button validation #590

Merged
merged 1 commit into from
Oct 31, 2019

Conversation

lauren-d
Copy link
Contributor

@lauren-d lauren-d commented Oct 28, 2019

Co-Authored-by: Johnny Mariéthoz [email protected]
Co-Authored-by: Lauren-D [email protected]

Why are you opening this PR?

How to test?

  • ./scripts/bootstrap & ./scripts/setup
  • login as librarian
  • go to document editor
  • import EAN from BNF, edit or create document

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?

@lauren-d lauren-d requested review from a team October 28, 2019 15:55
@lauren-d lauren-d self-assigned this Oct 28, 2019
@lauren-d lauren-d added the WIP label Oct 28, 2019
@lauren-d lauren-d force-pushed the lau-#1074-fix-editor-validation branch from 677aaa5 to 1624aa8 Compare October 29, 2019 08:17
@lauren-d lauren-d removed the WIP label Oct 29, 2019
@lauren-d lauren-d force-pushed the lau-#1074-fix-editor-validation branch from 1624aa8 to 2099a62 Compare October 29, 2019 12:40
* Enable/disable FormControl when a fieldset is show/hidden.
* Adapt json schema form and json schema documents.
* Fix save button disabled in document editor.
* Fix identifier subfield labelled with the same text.
* Close rero#556, Close rero#557

Co-Authored-by: Johnny Mariéthoz <[email protected]>
Co-Authored-by: Lauren-D <[email protected]>
@lauren-d lauren-d force-pushed the lau-#1074-fix-editor-validation branch from 2099a62 to 9e823eb Compare October 29, 2019 12:50
@lauren-d lauren-d requested a review from jma October 29, 2019 12:53
@iGormilhit iGormilhit requested a review from a team October 30, 2019 07:19
@jma jma merged commit 4fab090 into rero:dev Oct 31, 2019
@lauren-d lauren-d deleted the lau-#1074-fix-editor-validation branch January 28, 2020 14:29
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