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

[15.0][MIG] delivery_ups_oca #525

Closed
wants to merge 29 commits into from
Closed

[15.0][MIG] delivery_ups_oca #525

wants to merge 29 commits into from

Conversation

jaredkipe
Copy link

No description provided.

hbrunn and others added 29 commits August 22, 2022 17:14
@hbrunn @jaredkipe
[ADD] delivery_carrier_label_ups
d7f103b
@StefanRijnhart @jaredkipe
[RFR] Extract Test classes from Case classes for easier overriding
ced56a5
@astirpe @jaredkipe
[FIX] access to ir.config_parameter
393013e
@victoralmau @jaredkipe
[IMP] delivery_carrier_label_ups: black, isort, prettier
1e51d8f
@victoralmau @jaredkipe
[MIG] delivery_ups_oca: Migration to 13.0 + Refactor
0df1e4f
@oca-travis @jaredkipe
[UPD] Update delivery_ups_oca.pot
8241d87
@OCA-git-bot @jaredkipe
[UPD] README.rst
99a962d
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Prevent error when convert price in different…
2288f92 
… currency.
@OCA-git-bot @jaredkipe
delivery_ups_oca 13.0.1.0.1
10fe76a
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Define the total weight correctly in an order.
0356c4e
@OCA-git-bot @jaredkipe
delivery_ups_oca 13.0.1.0.2
68c444f
@victoralmau @jaredkipe
[REF] delivery_ups_oca: Code refactor.
0329a61
@oca-travis @jaredkipe
[UPD] Update delivery_ups_oca.pot
0b3dce4
@OCA-git-bot @jaredkipe
delivery_ups_oca 13.0.1.0.3
0c3e878
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Pevent errors in tests.
5beb375
@OCA-git-bot @jaredkipe
delivery_ups_oca 13.0.1.0.4
2845b2b
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Show the errors to avoid the error of access …
9155c26 
…to a non-existing key in the code.

TT37248
@OCA-git-bot @jaredkipe
delivery_ups_oca 13.0.1.0.5
96abb82
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Create label from create_shipping response to…
48de5b8 
… avoid error.

TT37248
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Set the correct URLs.
f298888
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Apply domain to ups_default_packaging_id fiel…
a5028be 
…d to show only UPS records.

TT37248
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Set the correct weight if we use packages.
af4e5dc 
TT37180
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Fix packages when package_ids is not set.
744da59 
TT37248
@victoralmau @jaredkipe
[FIX] delivery_ups_oca: Set correct ShipFrom (warehouse or company ad…
9fd6104 
…dress).
@victoralmau @jaredkipe
[IMP] delivery_ups_oca: Improve tests to allow inheritance.
24d0fa7 
TT37248
@jaredkipe
[UPD] Update delivery_ups_oca.pot
0c637b6
@OCA-git-bot @jaredkipe
delivery_ups_oca 13.0.2.0.0
01fb6c1
@jaredkipe
[MIG] delivery_ups_oca: to 15.0
43a2db3
@jaredkipe
[FIX] delivery_ups_oca: view domain, package_type, zpl extension
64bc9a5
@rousseldenis
Copy link
Sponsor Contributor

/ocabot migration delivery_ups_oca

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Oct 10, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 10, 2022
Open
31 tasks
@rousseldenis
Copy link
Sponsor Contributor

@jaredkipe Could you fix this ?

@sebalix sebalix changed the title MIG 15.0 delivery_ups_oca [15.0][MIG] delivery_ups_oca Dec 6, 2022
@victoralmau
Copy link
Member

Please, cherry-pick #578 to commit history

chandni299
chandni299 reviewed Jan 23, 2023
View reviewed changes
delivery_ups_oca/__manifest__.py
{
"name": "Delivery UPS OCA",
"summary": "Integrate UPS webservice",
"version": "13.0.2.0.0",
Copy link

Choose a reason for hiding this comment

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

We can change "version": "15.0.1.0.0"

@chandni299
Copy link

@jaredkipe Can you please look into the last comments/reviews?

@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 Jun 18, 2023
@wytzejan
Copy link
Member

@jaredkipe In case you continue working on this one I'd like to inform you about this: regarding UPS documentation https://www.ups.com/upsdeveloperkit/manageaccesskeys?loc=en_US it seems that OAuth is required now. I think this is good to know.

@HviorForgeFlow also made this comment in the pull request for v14: #639

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 25, 2023
@rousseldenis
Copy link
Sponsor Contributor

@jaredkipe Do you think you'll have time to fix this ?

@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 Nov 26, 2023
@github-actions github-actions bot closed this Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chandni299 chandni299 chandni299 left review comments

Assignees
No one assigned
Labels
migration stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.

None yet