-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[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
base: 18.0
Are you sure you want to change the base?
Conversation
Hey @slinkous this is ready for peer review. Thanks! |
There was a problem hiding this 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.
in Odoo. In the :guilabel:`Provider` field, select :guilabel:`FedEx` from the drop-down menu, if it | ||
is not already selected. | ||
|
||
The rest of the fields in this section are general to all shipping providers. For details on how to | ||
The rest of the fields in this section are general to all delivery providers. For details on how to | ||
fill them out, see the documentation on :doc:`third-party shippers <third_party_shipper>`. | ||
|
||
Fedex Configuration tab |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this to match a capitalization error in the UI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ope, yep, I see the picture now
A *developer key* is used to integrate a FedEx account with an external service, like the Odoo | ||
**Inventory** app. A *meter number* is a unique ID number used by FedEx to identify negotiated | ||
shipping rates for each account. | ||
After creating a developer account, navigate to the :guilabel:`My Projects` tab, and click |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is developer account distinct from the business account mentioned above? Could also use external link here, since it's a load-bearing step
<https://www.fedex.com/en-us/developer/web-services.html>`_. Then, click on the :guilabel:`FedEx Web | ||
Services` drop-down menu. | ||
On the :guilabel:`Tell us about your API needs` popup, select `Ships with FedEX and needs to | ||
integrate FedEx APIs into their system` in the :guilabel:`I work for a company that` drop-down. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
integrate FedEx APIs into their system` in the :guilabel:`I work for a company that` drop-down. | |
integrate FedEx APIs into their system` in the :guilabel:`I work for a company that:` drop-down. |
Docs task: https://www.odoo.com/odoo/project.task/4905285
Updating FedEx configuration doc to reflect updated UI/setup process.