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

Add totals for inventory and FMV to storage locations index page. #4520

Open
cielf opened this issue Jul 14, 2024 · 8 comments · May be fixed by #4768
Open

Add totals for inventory and FMV to storage locations index page. #4520

cielf opened this issue Jul 14, 2024 · 8 comments · May be fixed by #4768
Assignees

Comments

@cielf
Copy link
Collaborator

cielf commented Jul 14, 2024

Summary

Add grand totals for inventory and FMV (Fair Market Value) to storage locations index page

Why

Save a bit of time for the banks

Details

localhost:3000/storage_locations

Note that any filters must be taken into account. The grand total should be the grand total for the currently filtered situation.

Criteria for completion

-[ ] Storage Locations page has totals for inventory, FMV
-[ ] Tests to demonstrate that this works (for a case with multiple SLs, of course)

@cielf cielf added Help Wanted Groomed + open to all! Difficulty—Beginner labels Jul 14, 2024
@awwaiid awwaiid modified the milestone: Tasks 2024 Jul 28, 2024
@sunsheeppoplar
Copy link
Contributor

hey @cielf do you mind assigning this to me, please?

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label Aug 2, 2024
@cielf
Copy link
Collaborator Author

cielf commented Aug 2, 2024

Sure! (Welcome back?! Seems I haven't seen you for awhile!)

Copy link
Contributor

github-actions bot commented Sep 2, 2024

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

Copy link
Contributor

github-actions bot commented Sep 9, 2024

Automatically unassigned after 7 days of inactivity.

@cielf cielf added Help Wanted Groomed + open to all! and removed stale labels Sep 9, 2024
@Aaryanpal
Copy link
Contributor

Hey @cielf,
Please Assign this!

Thanks
@Aaryanpal

@cielf
Copy link
Collaborator Author

cielf commented Sep 23, 2024

Aren't you working on 4546?

@tatheerf02
Copy link
Contributor

Can I please take this on?

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label Nov 4, 2024
@cielf
Copy link
Collaborator Author

cielf commented Nov 4, 2024

Yes. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants