118 option wegboeken #122
Merged
118 option wegboeken #122
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Closed
SimplyPancake
commented
Oct 8, 2024
- Django admin: Add inline display of purchases in a writeoff order - Add toString for writeoff categories, to improve display in Django admin - Add a writeoff order detail page to view individual writeoff order details. - Add a list of writeoff orders to the drink orders page. - Increase max description length of a writeoff category - Actually display the writeoff category description on the writeoff button, in the order overview, and on the writeoff detail page - Move util to a model class method, as it makes more sense there. - Add a cancel button to the writeoff screen, to go back to the regular sales view. - Add a timer to write-off orders, just like regular orders, so they can be cancelled if a mistake is made - CSS tweaks to improve layout of the writeoff juliana frontend
Kurocon
approved these changes
Oct 11, 2024
|
I checked it out and I agree with the changes! Thanks for the 'small' changes ;) The changes make it more coherent, looks good! |
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please describe what your PR is fixing
Adds a write off option, where it can be enabled in the django admin per organisation.
When it is enabled, writeoff categories can be added in the admin panel of the organisation.
Then, bartenders are able to select 'write off' in the juliana POS, where they can put their orders on a category.
Then, in the sales overview, a tab has been added for the writeoff categories, where it shows how much (and which items) have been written off per category. A JSON export is also available.
Concretely, which issues does your PR solve?
#118
Does your PR include any django migrations?
yes!
PR does not include CSS changes