Skip to content

[17.0][ADD] Added a pos_multi_order_payment flow.#1339

Closed
kamladodiyaserpentcs wants to merge 2 commits into
OCA:17.0from
kamladodiyaserpentcs:17.0-multiple-payment
Closed

[17.0][ADD] Added a pos_multi_order_payment flow.#1339
kamladodiyaserpentcs wants to merge 2 commits into
OCA:17.0from
kamladodiyaserpentcs:17.0-multiple-payment

Conversation

@kamladodiyaserpentcs

@kamladodiyaserpentcs kamladodiyaserpentcs commented Mar 19, 2025

Copy link
Copy Markdown

No description provided.

@kamladodiyaserpentcs

kamladodiyaserpentcs commented Mar 19, 2025

Copy link
Copy Markdown
Author

@cvinh ,
Can you please review it?

@cvinh

cvinh commented Mar 19, 2025

Copy link
Copy Markdown
Contributor

This PR superseeds #1114

@kamladodiyaserpentcs kamladodiyaserpentcs force-pushed the 17.0-multiple-payment branch 5 times, most recently from 093f4df to b593553 Compare March 19, 2025 17:14
@kamladodiyaserpentcs

kamladodiyaserpentcs commented Mar 20, 2025

Copy link
Copy Markdown
Author

Hello @cvinh ,
Codecov
Can you please verify the screenshot? My code is 100% coverage.
Can you please review it?

@vehi-invitu

vehi-invitu commented Jul 9, 2025

Copy link
Copy Markdown

hi @kamladodiyaserpentcs, could you rebase your PR so we can test it with runboat, please?

@kamladodiyaserpentcs kamladodiyaserpentcs force-pushed the 17.0-multiple-payment branch 2 times, most recently from 52cd135 to 5e9cbc7 Compare July 9, 2025 05:23
@kamladodiyaserpentcs

kamladodiyaserpentcs commented Jul 9, 2025

Copy link
Copy Markdown
Author

Hello @vehi-invitu ,
Can you please review it?

vehi-invitu
vehi-invitu approved these changes Jul 9, 2025
View reviewed changes

@vehi-invitu vehi-invitu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you @kamladodiyaserpentcs.

@cvinh

cvinh commented Jul 10, 2025

Copy link
Copy Markdown
Contributor

Thank you @kamladodiyaserpentcs
Please rebase your branch with last 17.0
There should be only one commit in the PR

@kamladodiyaserpentcs

kamladodiyaserpentcs commented Jul 10, 2025

Copy link
Copy Markdown
Author

@cvinh ,
Can you please review it?

@github-actions

github-actions Bot commented Nov 9, 2025

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 9, 2025
@github-actions github-actions Bot closed this Dec 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@vehi-invitu vehi-invitu vehi-invitu approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kamladodiyaserpentcs @cvinh @vehi-invitu @etobella