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.

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

Cannot change calculator on existing shipping method #7657

Closed
Renatacumming opened this issue May 18, 2021 · 4 comments
Closed

Cannot change calculator on existing shipping method #7657

Renatacumming opened this issue May 18, 2021 · 4 comments

Comments

@Renatacumming
Copy link

Description

When a user tries to change an existing shipping method; wants to change calculator from Weights, to Flat Rate, and gets the 'opps, something went wrong' error. Have recreated this in the Aus Demo Hub with same result.

Expected Behavior

Should be able to change the calculator from one method to another method and save with no error

Actual Behaviour

When the user changes the calculator from one method to another method the 'oops, something went wrong' error appears

Steps to Reproduce

  1. Go into an existing shipping method.
  2. Try to change the calculator from one method to another method
  3. Click Update

Workaround

Create new shipping method and make old shipping method 'display - back office only'

bug-s3: a feature is broken but there is a workaround

@kirstenalarsen
Copy link
Contributor

kirstenalarsen commented May 18, 2021

Is coming up in Bugsnag -https://app.bugsnag.com/yaycode/openfoodnetwork/errors/60a34c07fbf6410007eb6401?event_id=60a34c07007c935b1e800000&i=sk&m=nw

I suspect this could be a regression in this release, perhaps to do with adjustments work @Matt-Yorkley ?

@filipefurtad0
Copy link
Contributor

filipefurtad0 commented May 18, 2021

This looks related and perhaps a duplicate of:
#6300

@Matt-Yorkley
Copy link
Contributor

Matt-Yorkley commented May 18, 2021

It doesn't seem to be related to adjustments (it's something to do with units and weights), and also doesn't look like it's new in this release...

@filipefurtad0
Copy link
Contributor

Confirmed duplicate of #6300, closing here.

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

No branches or pull requests

4 participants