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

Cookies for storing pagination settings | Non-Integrated eCR Viewer #2273

Open
2 tasks
gordonfarrell opened this issue Jul 29, 2024 · 2 comments · May be fixed by #2795
Open
2 tasks

Cookies for storing pagination settings | Non-Integrated eCR Viewer #2273

gordonfarrell opened this issue Jul 29, 2024 · 2 comments · May be fixed by #2795
Assignees
Labels
Streamline eCR Issues for Streamline eCR team to work on

Comments

@gordonfarrell
Copy link
Collaborator

gordonfarrell commented Jul 29, 2024

User Story

As somebody viewing the eCR Library table, I want my pagination choices to be retained when clicking into an eCR then returning to the eCR library, so that I don't have to constantly reset it to my desired settings.

Acceptance Criteria

  • Using cookies, pagination data is stored for a user
  • eCRs per page and the current page being viewed both persist after clicking into and returning from an eCR

Scenarios

Scenario 1

  • I navigate to the eCR Library and see a list of 25 eCRs
  • I want to view more eCRs per page, so I click the dropdown in the bottom right
  • I select 75 eCRs
  • The table updates to show 75 eCRs per page, and all other pagination values update properly (there should be fewer pages overall)
  • I click into an eCR to look at it
  • When I click the back button and return to the eCR Library it still shows me 75 per page

Scenario 2

  • I am viewing page 1 of the eCR Library with 50 eCRs per page and 200 total eCRs.
  • I see a display in the bottom left saying Showing 1-50 of 200 eCRs
  • I click the option for page 3 at the bottom
  • The display text now reads Showing 101-150 of 200 eCRs
  • I see eCRs 101-150 in my eCR Library
  • I click into an eCR to look at it
  • When I click the back button and return to the eCR Library it still shows eCRs 101-150

Design Notes

Link to Figma design in Dev mode
Link to Figma prototype

Screenshots below should not be used for working on this ticket - always refer to Figma design for most up-to-date designs:

Screenshot 2024-07-22 at 2.59.06 PM.png

@gordonfarrell gordonfarrell added the Streamline eCR Issues for Streamline eCR team to work on label Jul 29, 2024
@gordonfarrell gordonfarrell changed the title Copy of FRONTEND eCR Library Pagination Upgrade | Non-Integrated eCR Viewer Cookies for storing pagination settings | Non-Integrated eCR Viewer Jul 29, 2024
@emmastephenson
Copy link
Collaborator

@BobanL to double-check that this is already implemented.

@BobanL
Copy link
Collaborator

BobanL commented Oct 15, 2024

So part of this is done. eCRs per page is saved in local storage but the current page being viewed is not being saved.

@JNygaard-Skylight JNygaard-Skylight self-assigned this Oct 17, 2024
@JNygaard-Skylight JNygaard-Skylight linked a pull request Oct 18, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Streamline eCR Issues for Streamline eCR team to work on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants