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

FIX #28347 FIX #28962 FIX #29224 FIX #29035 #29303

Merged

Conversation

sonikf
Copy link
Contributor

@sonikf sonikf commented Apr 10, 2024

This feature is currently quite broken, first try to remedy.
Also in #29035 payment terms and other client attributes are carried over.

@eldy eldy added the PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) label Apr 10, 2024
@sonikf sonikf requested a review from eldy April 10, 2024 11:08
@sonikf sonikf marked this pull request as draft April 10, 2024 22:00
@sonikf sonikf changed the title FIX #28347 FIX #28962 FIX #29224 FIX #28347 FIX #28962 FIX #29224 FIX #29035 Apr 11, 2024
@sonikf sonikf marked this pull request as ready for review April 11, 2024 22:06
@eldy eldy added Discussion Some questions or discussions are opened and wait answers of author or other people to be processed PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) and removed PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) Discussion Some questions or discussions are opened and wait answers of author or other people to be processed labels Apr 12, 2024
Assign a subprice value of 0 to user selected lines without a vendor product price when creating a supplier order from a client order
@sonikf sonikf requested a review from eldy April 12, 2024 10:29
@eldy eldy merged commit 95aa03e into Dolibarr:19.0 Apr 12, 2024
4 checks passed
@sonikf sonikf deleted the sonikf-patch-supplier-order-from-client-order branch April 12, 2024 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants