Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[IMP] Inventory: Fedex Configuration update #13955

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

Closed
wants to merge 1 commit into from

Conversation

jero-odoo
Copy link
Contributor

Docs task: https://www.odoo.com/odoo/project.task/4905285

Updating FedEx configuration doc to reflect updated UI/setup process.

@jero-odoo jero-odoo self-assigned this Jul 9, 2025
@robodoo
Copy link
Collaborator

robodoo commented Jul 9, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team July 9, 2025 17:02
@jero-odoo jero-odoo requested a review from slinkous July 9, 2025 17:04
@jero-odoo
Copy link
Contributor Author

Hey @slinkous this is ready for peer review. Thanks!

Copy link
Contributor

@slinkous slinkous left a comment

Choose a reason for hiding this comment

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

This PR really delivers ;). Very minor suggestions.

@jero-odoo jero-odoo force-pushed the 18.0-fedex-setup-jero branch from 2a44fed to 6df0ac5 Compare July 10, 2025 12:59
@jero-odoo
Copy link
Contributor Author

Hey @Felicious this is ready for tech review (I tagged the expert in the task card as well)

Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

LGTM, just need two quick things:

  1. add the section the expert requested
  2. changing delivery connector back to shipping connector since that's how it's portrayed in the UI. (Changing shipping methods to delivery methods sounds good to me!)

@jero-odoo jero-odoo force-pushed the 18.0-fedex-setup-jero branch from 6df0ac5 to c200006 Compare July 17, 2025 17:07
@jero-odoo jero-odoo requested a review from Felicious July 17, 2025 17:07
@jero-odoo
Copy link
Contributor Author

@Felicious ready for another round

Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

LGTM @jero-odoo !

Approved with a single typo @robodoo delegate+

Co-authored-by: Zaz Linkous <[email protected]>

Co-authored-by: Felicia Kuan <[email protected]>
@jero-odoo jero-odoo force-pushed the 18.0-fedex-setup-jero branch from 0013f3f to a7fe3a3 Compare July 18, 2025 14:45
@jero-odoo
Copy link
Contributor Author

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Jul 18, 2025

@jero-odoo you can't review+.

@Felicious
Copy link
Contributor

Trying again. @jero-odoo

@robodoo delegate+

@jero-odoo
Copy link
Contributor Author

@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants