Skip to content

Product transaction history and undo button.#5

Open
ssiegel wants to merge 4 commits intomainfrom
add-produt-undo-routine
Open

Product transaction history and undo button.#5
ssiegel wants to merge 4 commits intomainfrom
add-produt-undo-routine

Conversation

@ssiegel
Copy link
Copy Markdown
Owner

@ssiegel ssiegel commented Oct 25, 2025

Bellow the stock items, show the last few product transactions.
The transactions lines can probably be similar to the stock lines in appearance.
Each line should have a undo button at the far right edge.

…etEntity grocy call, move all state independent fetch functions to grocy.ts, clean up, fmt
@tsanona
Copy link
Copy Markdown
Collaborator Author

tsanona commented Oct 2, 2025

progress on this made in fb39ef3

@ssiegel
Copy link
Copy Markdown
Owner

ssiegel commented Oct 25, 2025

Thank you, after testing I see the following issues:

  • The list of transactions seems to have arbitrary order, with the most recent transactions all over the place. It would be good to have the most recent transactions at the top of the list.
  • After undoing a transaction, the transaction disappears. It would be nice to keep it in the list struck out, like in Grocy itself.
  • When undoing a transaction, an update of the product page should be triggered, so the new state is reflected on screen without unnecessary delay.
  • The "Transaction" column should be left-aligned, the amount right-aligned.
  • A "Location" column might be useful in general and for the transfer-from/-to transactions in particular.

@ssiegel
Copy link
Copy Markdown
Owner

ssiegel commented Oct 25, 2025

Undoing a product-open transaction doesn't seem to work, at least if opening the product involved an automatic transfer to the consume location (i.e. the transaction consists of product-open, transfer_from, transfer_to). The selected line disappears from the list, but the transaction isn't actually reverted, the data on the product page doesn't change, and when viewing a different product and then returning to this product, the transaction line is there again.

@ssiegel ssiegel force-pushed the main branch 2 times, most recently from d5c1fda to 55d8a75 Compare February 22, 2026 19:04
@ssiegel
Copy link
Copy Markdown
Owner

ssiegel commented Mar 9, 2026

This was merged, minus the last commit “correctly refresh remote data after consume” because it causes a regression (stock not updating immediately after consume).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants