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

[IMP] delivery: improve delivery ux on sale order #28976

Closed
wants to merge 3 commits into from
Closed

[IMP] delivery: improve delivery ux on sale order #28976

wants to merge 3 commits into from

Conversation

Whenrow
Copy link
Contributor

@Whenrow Whenrow commented Nov 23, 2018 •
edited
Loading

This commit aims to improve the UX regarding the choice of delivery carrier from the sale orders.

  • Adding a new wizard to choose the carrier and check the estimated delivery fee
  • Invoicing policy adapts the delivery flow by adding or not the delivery fee on the sale order
  • Some UI features during the creation of the sale order

Task:1908654

@C3POdoo C3POdoo added the RD research & development, internal work label Nov 23, 2018
@Whenrow Whenrow force-pushed the master-ux-delivery-whe branch 4 times, most recently from bf93843 to 193588b Compare December 5, 2018 10:46
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Dec 5, 2018
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Jan 7, 2019
@Whenrow Whenrow force-pushed the master-ux-delivery-whe branch 3 times, most recently from 3403ffd to 7cc1ace Compare January 7, 2019 14:45
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Jan 7, 2019
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 10, 2019
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Jan 11, 2019
@Whenrow Whenrow force-pushed the master-ux-delivery-whe branch 3 times, most recently from 821a45f to ed386dc Compare January 14, 2019 08:02
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Jan 14, 2019
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 15, 2019
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jan 16, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 11, 2019
@robodoo
Copy link
Contributor

robodoo commented Feb 11, 2019

Linked pull request(s) odoo/enterprise#3587 not ready. Linked PRs are not staged until all of them are ready.

robodoo pushed a commit that referenced this pull request Feb 11, 2019
@Whenrow
[FIX] website_sale_delivery: remove unused variable
59e6071 
3bf8e75 changes the only call to
has_delivery to a check on another variable. This commit removes
the field from the model as it's now useless.

Task : 1908654

closes #28976
@robodoo robodoo added merging 👷 and removed merging 👷 CI 🤖 Robodoo has seen passing statuses labels Feb 11, 2019
@robodoo
Copy link
Contributor

robodoo commented Feb 11, 2019

Staging failed: ci/runbot (view more at http://runbot.odoo.com/runbot/build/446067)

@sle-odoo
Copy link
Contributor

robodoo retry

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses r+ 👌 and removed error 🙅 labels Feb 11, 2019
robodoo pushed a commit that referenced this pull request Feb 11, 2019
@Whenrow
[FIX] website_sale_delivery: remove unused variable
a1f22d5 
3bf8e75 changes the only call to
has_delivery to a check on another variable. This commit removes
the field from the model as it's now useless.

Task : 1908654

closes #28976
@robodoo robodoo added merging 👷 and removed merging 👷 CI 🤖 Robodoo has seen passing statuses labels Feb 11, 2019
@robodoo
Copy link
Contributor

robodoo commented Feb 11, 2019

Staging failed: timed out (>120 minutes)

@Whenrow
Copy link
Contributor Author

Whenrow commented Feb 12, 2019

robodoo retry

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses r+ 👌 and removed error 🙅 labels Feb 12, 2019
@robodoo
Copy link
Contributor

robodoo commented Feb 12, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 12, 2019
@Whenrow Whenrow deleted the master-ux-delivery-whe branch February 12, 2019 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amoyaux amoyaux amoyaux left review comments

@sle-odoo sle-odoo sle-odoo left review comments

@sswapnesh sswapnesh sswapnesh left review comments

@pimodoo pimodoo pimodoo left review comments

Assignees
No one assigned
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Whenrow @sle-odoo @robodoo @amoyaux @sswapnesh @pimodoo @C3POdoo