human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

Add packages (not packs) as an option for the distribution export (from Nicole D, 20240807 stakeholder meeting)

Open cielf opened this issue 1 year ago • 7 comments

Summary

Add the ability to have packages (not packs) as an option for the distribution export.

Why?

Request from a bank.

Details

As an option on organization, add a flag that indicates that packages should be added to the distribution export (add to both edit and view) When that flag is true, the distribution export will show an additional column for each item showing the packages (items / number of items per package for that item). These should be interleaved with the current item columns

Criteria for completion

  • [ ] behaviour as above
  • [ ] automated tests to support the behaviour ( for both true and false values of the flag)

cielf avatar Sep 30 '24 02:09 cielf

Hey can i try this

Naraveni avatar Nov 28 '24 03:11 Naraveni

Please do.

cielf avatar Nov 28 '24 13:11 cielf

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.

github-actions[bot] avatar Dec 29 '24 00:12 github-actions[bot]

Automatically unassigned after 7 days of inactivity.

github-actions[bot] avatar Jan 06 '25 00:01 github-actions[bot]

I may be in over my head here.... but would it be ok if I give this a shot?

pshong79 avatar Feb 02 '25 03:02 pshong79

Sure!

cielf avatar Feb 02 '25 04:02 cielf

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.

github-actions[bot] avatar Mar 05 '25 00:03 github-actions[bot]

I can take this one.

brodyf42 avatar Sep 13 '25 19:09 brodyf42

@cielf, if for some reason the number of packages does not work out to a whole number, how would you like the data displayed?

brodyf42 avatar Sep 19 '25 22:09 brodyf42

@brodyf42 -- I would think that a float to 2 decimal places would work.

cielf avatar Sep 22 '25 14:09 cielf

@brodyf42 -- I would think that a float to 2 decimal places would work.

Updates for this have been pushed and are now included in the PR.

brodyf42 avatar Sep 23 '25 01:09 brodyf42

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.

github-actions[bot] avatar Oct 24 '25 00:10 github-actions[bot]

I see this got marked stale. The PR has been approved and is passing tests. I think it's good to merge. We walked through the tests together in one of the Sunday meetings about a month ago now.

brodyf42 avatar Oct 27 '25 23:10 brodyf42