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

circulation policy: ignore settings when deleting a policy #672

Merged
merged 1 commit into from
Dec 17, 2019

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Dec 13, 2019

Co-Authored-by: Aly Badr aly.badr@rero.ch

Why are you opening this PR?

How to test?

  • create a new circulation policy with settings.
  • you should be able to delete it.

Code review check list

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

@BadrAly BadrAly self-assigned this Dec 13, 2019
@AoNoOokami
Copy link
Contributor

AoNoOokami commented Dec 13, 2019

Actually, the issue related to this PR comes from rero-ils-ui: rero/rero-ils-ui#76. The issue mentioned in your commit message is this one: #76. Should we mention it this way in commit message: rero/rero-ils-ui#76 ?

* Fixes an issue when cipo settings blocked its deletion.
* Closes rero/rero-ils-ui#76

Co-Authored-by: Aly Badr <aly.badr@rero.ch>
@BadrAly BadrAly force-pushed the baa-#1215-cipo-settings branch from b75d134 to e2f8963 Compare December 13, 2019 14:09
@BadrAly
Copy link
Author

BadrAly commented Dec 13, 2019

Actually, the issue related to this PR comes from rero-ils-ui: rero/rero-ils-ui#76. The issue mentioned in your commit message is this one: #76. Should we mention it this way in commit message: rero/rero-ils-ui#76 ?

@AoNoOokami Done

@BadrAly BadrAly requested a review from iGormilhit December 16, 2019 09:40
@BadrAly BadrAly merged commit 0f22fe6 into rero:dev Dec 17, 2019
@BadrAly BadrAly deleted the baa-#1215-cipo-settings branch January 27, 2020 10:57
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.

4 participants