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: fix request with different locations #2260

Merged
merged 1 commit into from
Aug 9, 2021

Conversation

jma
Copy link
Contributor

@jma jma commented Aug 2, 2021

Checks the owning location allow_request property to decide if an item can be requested.

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch

Code review check list

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

@github-actions github-actions bot added the f: circulation Concerns the circulation interface or backend label Aug 2, 2021
@jma jma force-pushed the maj-fix-requests branch 2 times, most recently from ae41fd9 to 53828f1 Compare August 4, 2021 07:39
@jma jma marked this pull request as ready for review August 4, 2021 08:45
@@ -147,6 +147,7 @@ def test_receive_regular_issue_api(
'api_holding.receive_regular_issue',
url_data=dict(holding_pid=holding.pid)
)
print(data)
Copy link
Contributor

Choose a reason for hiding this comment

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

could be removed

* Checks the owning location `allow_request` property to decide if an item can be requested.
* Closes rero#2234.
* Logs an error and do a db rollback for holding views.
* Fixes tests that failed every two monthes.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma jma merged commit c8c58d6 into rero:dev Aug 9, 2021
@iGormilhit iGormilhit added this to the v1.4.7 milestone Sep 21, 2021
@jma jma deleted the maj-fix-requests branch January 13, 2022 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: circulation Concerns the circulation interface or backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Requests for patrons by librarians are impossible with librarian accounts of a specific library
5 participants