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

In new purchase, Category purchase values must be non-negative

Open cielf opened this issue 9 months ago • 7 comments

Summary

The category purchase values in new purchase / edit purchase should be constrained to be non-negative

Why?

Hygiene

Details

To see this, sign in as [email protected], then click "Purchases" and "New Purchase" in the left-hand menu.
Enter enough information to be able to save, including at least one negative number in the "purchase total for..." (still have the purchase total add up to the 4 category totals. Save.

It allows it! And shouldn't.

Put in checks so that none of the 'Purchase Total for..." fields can be negative.

Criteria for completion

  • [ ] behaviour as described above
  • [ ] tests to confirm behaviour as described above.

cielf avatar Mar 02 '25 16:03 cielf

I can work on this issue.

jahuang415 avatar Mar 06 '25 19:03 jahuang415

Thank you!

cielf avatar Mar 06 '25 19:03 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 Apr 07 '25 00:04 github-actions[bot]

@cielf Hi! I noticed this issue hasn’t had recent activity. I’m interested in working on this for a class project. If the original contributor is no longer working on it, I’d be happy to take it over.

boydstonl avatar Apr 09 '25 14:04 boydstonl

@boydstoni -- The changes that are left are essentially just changing the name of a test and removing a comment. If you want something that's still simple, but where you would make a visible difference, may I suggest #5063 -- it's just changing the styling on some buttons. (if you do -- comment on it and I will assign you).

cielf avatar Apr 09 '25 19:04 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 May 11 '25 00:05 github-actions[bot]

Automatically unassigned after 7 days of inactivity.

github-actions[bot] avatar May 18 '25 00:05 github-actions[bot]

i can work on this issue, please assign this to me

harshs1205 avatar Jul 18 '25 19:07 harshs1205

@harsh1205 Will do! If you look at the PR that was submitted - you can see that it really only needs a couple of very small changes to make it pass. (We just haven't gotten around to it)

cielf avatar Jul 19 '25 00:07 cielf