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.

Sign up for GitHub

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

Jump to bottom

[16.0][MIG] stock_landed_costs_delivery: Migration to version 16.0 #1559

Merged
merged 8 commits into from
May 29, 2024
Merged

[16.0][MIG] stock_landed_costs_delivery: Migration to version 16.0 #1559

merged 8 commits into from
May 29, 2024

Conversation

carolinafernandez-tecnativa
Copy link
Contributor

@carolinafernandez-tecnativa carolinafernandez-tecnativa commented Apr 2, 2024 •
edited
Loading

@carolinafernandez-tecnativa carolinafernandez-tecnativa marked this pull request as ready for review April 2, 2024 14:37
@pedrobaeza
Copy link
Member

/ocabot migration stock_landed_costs_delivery

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Apr 2, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 2, 2024
Open
71 tasks
victoralmau and others added 8 commits May 24, 2024 15:37
@victoralmau @carolinafernandez-tecnativa
[ADD] stock_landed_costs_delivery: Auto create cost line at destinati…
a57c29e 
…on with the

shipping information of the purchase order.
TT30782

[UPD] Update stock_landed_costs_delivery.pot

[UPD] README.rst
@victoralmau @carolinafernandez-tecnativa
[MIG] stock_landed_costs_delivery: Migration to 15.0
37fe085 
TT41587
@victoralmau @carolinafernandez-tecnativa
[MIG] stock_landed_costs_delivery: Refactor (v14.0)
9cbfdfa 
TT45656
@victoralmau @carolinafernandez-tecnativa
[IMP] stock_landed_costs_delivery: Auto-validate Landed Costs when li…
bfc6aec 
…nked purchase pickings are done

TT45656
@victoralmau @carolinafernandez-tecnativa
[IMP] stock_landed_costs_delivery: Allow create stock.landed.cost wit…
a9e43b0 
…hout advanced permissions

TT45656
@victoralmau @carolinafernandez-tecnativa
[FIX] stock_landed_costs_delivery: Filtering the correct Landed Cost
9792d71 
It is necessary to filter the correct Landed Cost (from the correct picking)
to set the cost lines.

Example use case:
- PO without carrier.
- Picking A without carrier done > LC A (draft) without cost lines.
- Picking B with carrier done > LC B (done) with cost lines.

TT45656
@carolinafernandez-tecnativa
[IMP] stock_landed_costs_delivery: pre-commit stuff
b5b5aa3
@carolinafernandez-tecnativa
[MIG] stock_landed_costs_delivery: Migration to 16.0
5731d17
@carolinafernandez-tecnativa carolinafernandez-tecnativa force-pushed the 16.0-mig-stock_landed_costs_delivery branch 2 times, most recently from c544017 to 5731d17 Compare May 29, 2024 08:49
@carolinafernandez-tecnativa
Copy link
Contributor Author

ping @pedrobaeza

pedrobaeza
pedrobaeza approved these changes May 29, 2024
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1559-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 29, 2024
@OCA-git-bot
Merge PR #1559 into 16.0
775a6a9 
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1559-by-pedrobaeza-bump-nobump.

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.

@pedrobaeza
Copy link
Member

Greenified in #1605

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1559-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 0a6348d into OCA:16.0 May 29, 2024
9 of 11 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 16.0-mig-stock_landed_costs_delivery branch May 29, 2024 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victoralmau victoralmau victoralmau approved these changes

@pedrobaeza pedrobaeza pedrobaeza approved these changes

@Christian-RB Christian-RB Christian-RB approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.
5 participants
@carolinafernandez-tecnativa @pedrobaeza @OCA-git-bot @victoralmau @Christian-RB