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

Price#edit: Neither is current country selected nor can I set a new country #1860

Closed
robinboening opened this issue Apr 23, 2017 · 2 comments

Comments

@robinboening
Copy link
Contributor

Steps to reproduce

Case 1: Initially created price

  1. Create a product
  2. Go to /admin/products/xyz/prices
  3. Click on the icon to edit the price

Case 2: Additional price with certain country

  1. Go to /admin/products/xyz/prices
  2. Create another price and choose "Germany" as the country.
  3. Click on the icon to edit this price

Expected behavior

In Case 1:

  • I should be able to change the value in the selectbox for countries

In Case 2:

  • I should be able to change the value in the selectbox for countries
  • If I created a price for a certain country I should see that country as the selected value.

Actual behavior

In Case 1:

  • The country selectbox is disabled which prevents me from choosing a country.

In Case 2:

  • The country selectbox is disabled which prevents me from choosing a country.
  • No country is selected. It always shows the first select-value "any country" even if its not the actual value.

System configuration

Solidus Version: 2.3.0.alpha

Extensions in use: None

@jhawthorn
Copy link
Contributor

In Case 1:

I should be able to change the value in the selectbox for countries
In Case 2:

I should be able to change the value in the selectbox for countries

This is intended behaviour. A new price should be created for different countries

No country is selected. It always shows the first select-value "any country" even if its not the actual value.

This does seem to be a bug 🐛 . Taking a look

@jhawthorn
Copy link
Contributor

Bug should be fixed by #2266

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

2 participants