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

[16.0][FIX] delivery_postlogistics: if street2 is defined, show it after street on shipping label #669

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

sonhd91
Copy link

@sonhd91 sonhd91 commented Jul 4, 2023

Currently, street 2 is printed before street 1 on the shipping label, this fix suggest that street1 will shown in front of street 2

image
image

@sonhd91 sonhd91 changed the title [FIX]fix index street value on shipping label [16.0][FIX] delivery_postlogistics: if street2 is defined, show it after street on shipping label Jul 4, 2023
@simahawk
Copy link
Contributor

simahawk commented Jul 18, 2023

@sonhd91 can you pls rewrite the commit msg as [fix] delivery_postlogistics: address street order or similar?

@sonhd91 sonhd91 force-pushed the 16.0-fix_street_value_on_label branch from 97ea982 to 471d4f7 Compare July 18, 2023 09:10
@sonhd91
Copy link
Author

sonhd91 commented Jul 18, 2023

@sonhd91 can you pls rewrite the commit msg as [fix] delivery_postlogistics: address street order or similar?

Branch updated new commit msg

@simahawk
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-669-by-simahawk-bump-patch, awaiting test results.

@simahawk
Copy link
Contributor

If we have the same issue on 14 and 15 it would be nice if you could backport it ;)

@OCA-git-bot OCA-git-bot merged commit e6720bc into OCA:16.0 Jul 18, 2023
5 of 6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a6456f4. 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.

5 participants