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

patrons: display checkout history for patron #720

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Jan 28, 2020

  • Adds history tab for logged in patrons to display history of last 6 months.

Co-Authored-by: Aly Badr [email protected]

Why are you opening this PR?

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?

@BadrAly BadrAly added the WIP label Jan 28, 2020
@BadrAly BadrAly force-pushed the baa-patron-profile branch 4 times, most recently from 8fbd7b4 to e0038f8 Compare January 28, 2020 14:56
@BadrAly BadrAly removed the WIP label Jan 28, 2020
@BadrAly BadrAly self-assigned this Jan 28, 2020
@BadrAly BadrAly force-pushed the baa-patron-profile branch from e0038f8 to 986f567 Compare January 31, 2020 12:03
@BadrAly BadrAly force-pushed the baa-patron-profile branch 2 times, most recently from 892083e to 5a94970 Compare February 5, 2020 07:26
@Garfield-fr Garfield-fr added this to the v0.6.0 milestone Feb 6, 2020
@BadrAly BadrAly force-pushed the baa-patron-profile branch 2 times, most recently from 0839018 to 1c3225e Compare February 6, 2020 15:24
* Adds history tab for logged in patrons to display history of last 6 months.

Co-Authored-by: Aly Badr <[email protected]>
@BadrAly BadrAly merged commit c072503 into rero:dev Feb 12, 2020
@BadrAly BadrAly deleted the baa-patron-profile branch November 5, 2020 13:34
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.

5 participants