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

[IMP] pos_cash_move_reason : display cash moves on PoS session summary #742

Conversation

legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain commented Dec 24, 2021

Add in the pos session view, a new part to display all the cash moves done on the current session.

this makes it easier to check, without having to click on each payment method

pos_session_move_list_form

CC : @quentinDupont

apps/deck/#/board/144/card/2075

@legalsylvain legalsylvain force-pushed the 12.0-IMP-pos_cash_move_reason-display-cash-moves branch 2 times, most recently from 5925364 to 480c0ba Compare December 24, 2021 11:09
@legalsylvain legalsylvain added this to the 12.0 milestone Mar 6, 2022
@legalsylvain
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

@legalsylvain The merge process could not start, because command git clone --quiet --reference /app/run/.cache/oca-mqt/github.com/oca/pos --branch 12.0 -- https://github.com/OCA/pos /tmp/tmpjaq4ugfb failed with output:

fatal: unable to access 'https://github.com/OCA/pos/': Failed to connect to github.com port 443: Connection timed out

@legalsylvain
Copy link
Contributor Author

/ocabot merge minor

better ?

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-742-by-legalsylvain-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Mar 6, 2022
Signed-off-by legalsylvain
@OCA-git-bot
Copy link
Contributor

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-742-by-legalsylvain-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@AntoniRomera
Copy link

You can check this PR to make this improvement in verison 13.0 and 14.0.

@legalsylvain
Copy link
Contributor Author

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 12.0.

@OCA-git-bot OCA-git-bot force-pushed the 12.0-IMP-pos_cash_move_reason-display-cash-moves branch from 480c0ba to fc5439e Compare April 25, 2022 12:21
@legalsylvain
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-742-by-legalsylvain-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Apr 25, 2022
Signed-off-by legalsylvain
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 12.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 12.0-ocabot-merge-pr-742-by-legalsylvain-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 60daf35 into OCA:12.0 Apr 25, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at fc6bf3d. Thanks a lot for contributing to OCA. ❤️

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

Successfully merging this pull request may close these issues.

None yet

4 participants