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 location editor button validation #592

Merged
merged 1 commit into from
Oct 31, 2019

Conversation

lauren-d
Copy link
Contributor

Co-Authored-by: Lauren-D [email protected]

Why are you opening this PR?

How to test?

  • ./scripts/bootstrap & ./scripts/setup
  • login as librarian
  • add new location for library 3 ( ~/admin/records/locations/new?library=3)

note : clear cache of browser maybe is needed

Code review check list

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

* Add default return value to location async validator.
* Fix save button disabled in location editor.
* Closes rero#562.

Co-Authored-by: Lauren-D <[email protected]>
@lauren-d lauren-d requested review from a team October 30, 2019 11:01
@lauren-d lauren-d self-assigned this Oct 30, 2019
@lauren-d lauren-d added WIP and removed WIP labels Oct 30, 2019
@jma jma merged commit 6381745 into rero:dev Oct 31, 2019
@lauren-d lauren-d deleted the lau-#1114-fix-location-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