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: display pickup location name in patron profile #406

Merged
merged 1 commit into from Oct 29, 2020

Conversation

zannkukai
Copy link
Contributor

@zannkukai zannkukai commented Oct 29, 2020

When a request is available for pickup, the location name was the short
name. This PR corrects this problem: now the label display is the
'pickup_name' field from location.

Authored-by: Renaud Michotte renaud.michotte@gmail.com

Why are you opening this PR?

Closes rero/rero-ils#1300

How to test?

Before
image

After
image

Code review check list

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

@zannkukai zannkukai self-assigned this Oct 29, 2020
When a request is available for pickup, the location name was the short
name. This PR corrects this problem: now the label display is the
'pickup_name' field from location.

Closes rero/rero-ils#1300

Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai force-pushed the zan-#1822-pickup-location-name branch from dae6f9e to 987b91e Compare October 29, 2020 12:52
@zannkukai zannkukai added this to the v0.14.0 (rero-ils) milestone Oct 29, 2020
@zannkukai zannkukai merged commit d74e490 into rero:v0.14 Oct 29, 2020
@zannkukai zannkukai deleted the zan-#1822-pickup-location-name branch October 29, 2020 17:11
@iGormilhit iGormilhit added the f: circulation Concerns the circulation interface or backend label Jan 14, 2021
@iGormilhit iGormilhit added this to New in RERO ILS PRs via automation Jan 14, 2021
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
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants