-
-
Notifications
You must be signed in to change notification settings - Fork 492
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
[RFG 2023] Move Purchases card from dashboard to Purchases | Summary #3736
Labels
Ruby for Good 2023 DC
Created for Ruby for Good 2023 DC
Milestone
Comments
@awwaiid grabbing this one! |
rae-stanton
added a commit
that referenced
this issue
Jul 29, 2023
Fixes #3736 These changes include: - Removing Purchase Summary card from user's dashboard - Adds new purchase summary page & item in navigation - Changes to updated card styling/formatting - Adds a shared date filter card partial in shared directory - Adds a shared card partial for use throughout in the shared directory
rae-stanton
added a commit
that referenced
this issue
Jul 29, 2023
This reverts commit cd08a7b. :wq#
rae-stanton
added a commit
that referenced
this issue
Jul 29, 2023
Fixes #3736 These changes include: - Removing Purchase Summary card from user's dashboard - Adds new purchase summary page & item in navigation - Changes to updated card styling/formatting - Adds a shared date filter card partial in shared directory - Adds a shared card partial for use throughout in the shared directory
rae-stanton
added a commit
that referenced
this issue
Jul 29, 2023
Fixes #3736 These changes include: Needed to change the dashboard to not contain any information about the purchases summary - this has now been moved to the new purchases summary page within the nav bar. Filtering has also been added to said page, and filter partials and the card have been added to a shared area for ease of access and consistency. Tests now include logic for logged in/out & more as well.
rae-stanton
added a commit
that referenced
this issue
Jul 30, 2023
Fixes #3736 These changes include: Needed to change the dashboard to not contain any information about the purchases summary - this has now been moved to the new purchases summary page within the nav bar. Filtering has also been added to said page, and filter partials and the card have been added to a shared area for ease of access and consistency. Tests now include logic for logged in/out & more as well.
rae-stanton
added a commit
that referenced
this issue
Jul 30, 2023
Fixes #3736 These changes include: Needed to change the dashboard to not contain any information about the purchases summary - this has now been moved to the new purchases summary page within the nav bar under `reporting and auditing`. Filtering has also been added to said page, and filter partials and the card have been added to a shared area for ease of access and consistency. Page tests added for new content.
awwaiid
added a commit
that referenced
this issue
Jul 30, 2023
…rm-dash Move purchase from dashboard to standalone report [#3736]
PR done -- Overall QA and follow-up tickets will be done as part of an epic Dashboard review. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
Move the Purchases functionality from the dashboard to the left hand menu
Why
Part of Dashboard rework
Details
We are talking about the card accessed from the dashboard as "Purchases"
A view containing this info will be be accessed via the left hand menu by clicking "Reporting & Auditing" then "Purchases Summary"
This will require a new controller and view
The card will be removed from the dashboard
Note that we'll need the filtering functionality in the new report
Other things you should know
We are grouping a number of dashboard-related changes together and will merge these individual changes into a long lived dashboard-related branch. We will merge this long lived branch into main when we are ready to release the whole suite.
We may want to make the filtering functionality a common element between all of these separate reports (if it isn't already)
Criteria for completion
The text was updated successfully, but these errors were encountered: