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: improve displayed patron data #419

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

zannkukai
Copy link
Contributor

Depending where the card component is sued, the link on the patron name
should be different :

  • on the checkin interface : point to the patron circulation view
  • on the patron circulation view : point to the patron detail view

Closes rero/rero-ils#1378

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

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

Dependencies

My PR depends on rero-ils#<xx>'s PR(s):

  • rero/rero-ils#

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

Code review check list

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

@zannkukai zannkukai added this to the v0.14.0 milestone Nov 6, 2020
@zannkukai zannkukai self-assigned this Nov 6, 2020
@zannkukai zannkukai force-pushed the zan-#1837-adapt-card-component branch from 8b4549a to 6a0a277 Compare November 6, 2020 10:59
@zannkukai zannkukai marked this pull request as ready for review November 6, 2020 13:18
@iGormilhit iGormilhit added the f: circulation Concerns the circulation interface or backend label Nov 9, 2020
@zannkukai zannkukai force-pushed the zan-#1837-adapt-card-component branch from 6a0a277 to 81851d6 Compare November 9, 2020 09:02
@iGormilhit iGormilhit changed the title circulation: update link on card component circulation: improve displayed patron data Nov 9, 2020
* Adapts the link on the patron name depending where the card component
  is used:
  * In the checkin interface: point to the patron circulation view.
  * In the patron circulation view: point to the patron detail view.
* Displays the patron age instead of birth date, to identify children
  easily + easter eggs.

Closes rero/rero-ils#1378

Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai force-pushed the zan-#1837-adapt-card-component branch from 81851d6 to 353c5e5 Compare November 9, 2020 10:38
Copy link
Contributor

@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.

@zannkukai zannkukai merged commit 2d270d2 into rero:dev Nov 9, 2020
@zannkukai zannkukai deleted the zan-#1837-adapt-card-component branch November 9, 2020 11:54
@iGormilhit iGormilhit added the f: public ui Public interface, as opposed to the professional interface label Nov 12, 2020
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 f: public ui Public interface, as opposed to the professional interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In the check-in form, the patron information can be more useful (link + age).
3 participants