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] delivery_tnt_oca #649

Closed
wants to merge 41 commits into from
Closed

[16.0][MIG] delivery_tnt_oca #649

wants to merge 41 commits into from

Conversation

hildickethan
Copy link
Member

@hildickethan hildickethan commented May 25, 2023 •
edited
Loading

  • Migration to 16.0
  • Added PDF label format in seperate commit, I made it based on the ZPL one, not 1:1 but close enough
  • Added multiple package support for both reports, now using the weight returned for the piece rather than the picking weight. I did this trying to maintain the sparse fields and with minimal changes, so I saved the multi-piece data in a string and then split it in the report
  • Added Manifest, International manifest and Consignment note reports by @miguel-S73

@miguel-S73 miguel-S73 force-pushed the 16.0-mig-delivery_tnt_oca branch 2 times, most recently from 21dcb7e to 1de2c1e Compare June 6, 2023 08:00
@RabbitJon-S73 RabbitJon-S73 force-pushed the 16.0-mig-delivery_tnt_oca branch 2 times, most recently from 37830fa to e4ac7b6 Compare June 22, 2023 09:42
@RabbitJon-S73
Copy link
Contributor

I changed message when for some reason TNT server is not reachable to specify that is a TNT fail instead Odoo's fail

@miguel-S73 miguel-S73 force-pushed the 16.0-mig-delivery_tnt_oca branch 4 times, most recently from e692685 to 66a7b53 Compare June 30, 2023 11:35
@rousseldenis
Copy link
Contributor

/ocabot migration delivery_tnt_oca

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jun 30, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 30, 2023
Open
32 tasks
@hildickethan hildickethan force-pushed the 16.0-mig-delivery_tnt_oca branch 2 times, most recently from 9280edb to 18b8d58 Compare July 3, 2023 13:31
@hildickethan hildickethan force-pushed the 16.0-mig-delivery_tnt_oca branch 3 times, most recently from 555e4eb to 048d861 Compare July 13, 2023 15:48
oca-ci and others added 16 commits October 16, 2023 09:42
@hildickethan
[UPD] Update delivery_tnt_oca.pot
5c54e79
@weblate @hildickethan
Update translation files
08a0fd8 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: delivery-carrier-14.0/delivery-carrier-14.0-delivery_tnt_oca
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-14-0/delivery-carrier-14-0-delivery_tnt_oca/
@victoralmau @hildickethan
[FIX] delivery_tnt_oca: Set correct weight if not using packages
7378f38 
TT40767
@victoralmau @hildickethan
[FIX] delivery_tnt_oca: Set correct dimensions from packages
ea5aba2 
TT40767
@victoralmau @hildickethan
[FIX] delivery_tnt_oca: Set correct sequenceNumbers value from labels.
69e68f1 
TT40767
@OCA-git-bot @hildickethan
delivery_tnt_oca 14.0.1.1.2
4d8a76d
@victoralmau @hildickethan
[IMP] delivery_tnt_oca: Add the tnt_service_option field in the carri…
42bd316 
…er to send that value to the API.

TT40767
@hildickethan
[UPD] Update delivery_tnt_oca.pot
5486ba8
@OCA-git-bot @hildickethan
delivery_tnt_oca 14.0.1.2.0
1ab6104
@weblate @hildickethan
Update translation files
f6c7416 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: delivery-carrier-14.0/delivery-carrier-14.0-delivery_tnt_oca
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-14-0/delivery-carrier-14-0-delivery_tnt_oca/
@victoralmau @hildickethan
[IMP] delivery_tnt_oca: Set minimum dimensions of 0.1
af3a27d 
TT40767
@OCA-git-bot @hildickethan
delivery_tnt_oca 14.0.1.2.1
eb7b3d8
@victoralmau @hildickethan
[IMP] delivery_tnt_oca: Remove unnecessary BOOK tag from ACTIVITY
9c4784d 
TT40767
@OCA-git-bot @hildickethan
delivery_tnt_oca 14.0.1.2.2
4a22021
@pedrobaeza @hildickethan
[FIX] delivery_tnt_oca: Incorrect method name
177de03 
Fixes OCA#605
@OCA-git-bot @hildickethan
delivery_tnt_oca 14.0.1.2.3
05e5565
@github-actions GitHub Actions
Copy link

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 Feb 18, 2024
@github-actions github-actions bot closed this Mar 24, 2024
@pedrobaeza pedrobaeza reopened this Mar 24, 2024
@pedrobaeza pedrobaeza closed this Mar 24, 2024
@PlantBasedStudio
Copy link

Hello,
I'd like to know the details that caused this RP to fail and, if possible, try it again:
Can it be put into an open state again?

@hildickethan-S73 can we have some information?

Were the tests good?

@hildickethan
Copy link
Member Author

Hello, I'd like to know the details that caused this RP to fail and, if possible, try it again: Can it be put into an open state again?

@hildickethan-S73 can we have some information?

Were the tests good?

From what I remember this migration worked fine, it's just that the only client we had that used this module stopped using it so when the branch went stale I just let it close. I didn't delete the branch in case it was needed again, if you need it I suggest cherry-picking off https://github.com/Studio73/delivery-carrier/tree/16.0-mig-delivery_tnt_oca and opening a new PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
migration needs review stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.