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

items: fix loans circulation dump #2210

Merged
merged 1 commit into from
Aug 11, 2021
Merged

Conversation

zannkukai
Copy link
Contributor

Adds pickup location specific name into loan data dumped for
circulation.

Co-Authored-by: Renaud Michotte [email protected]

Code review check list

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

@zannkukai zannkukai requested a review from Garfield-fr July 20, 2021 14:35
@zannkukai zannkukai self-assigned this Jul 20, 2021
@zannkukai zannkukai requested a review from benerken July 20, 2021 14:35
@github-actions github-actions bot added the f: circulation Concerns the circulation interface or backend label Jul 20, 2021
Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

Commit message approved.

@iGormilhit iGormilhit added this to the v1.5.0 milestone Jul 22, 2021
@iGormilhit iGormilhit modified the milestones: v1.5.0, v1.4.6 Jul 22, 2021
Adds pickup location specific name into loan data dumped for
circulation.

Co-Authored-by: Renaud Michotte <[email protected]>
@zannkukai zannkukai force-pushed the zan-fix-issue-2207 branch from 1b05061 to cb73901 Compare July 22, 2021 06:44
@pronguen pronguen merged commit ad9651a into rero:dev Aug 11, 2021
@zannkukai zannkukai deleted the zan-fix-issue-2207 branch August 16, 2021 11:53
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.

5 participants