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

Clicking save on variant doesn't update main price #30

Closed
bstocks opened this issue Apr 13, 2014 · 2 comments
Closed

Clicking save on variant doesn't update main price #30

bstocks opened this issue Apr 13, 2014 · 2 comments

Comments

@bstocks
Copy link
Contributor

bstocks commented Apr 13, 2014

When updating the price of a selected variant, the main price doesn't change as soon as you click the save button.

To reproduce:

  • Select and open a variant
  • Change Price
  • Click "Save"
  • Note that while the variant price changed, the big price did not. Only when you click the variant bar again does the price update.
@bstocks bstocks added bug and removed bug labels Apr 13, 2014
@bstocks
Copy link
Contributor Author

bstocks commented Jun 19, 2014

This appears to be fixed now.

@bstocks bstocks closed this as completed Jun 19, 2014
@aaronjudd
Copy link
Contributor

ah yes, I did work on this ;-)

cmbirk pushed a commit to cmbirk/reaction that referenced this issue Aug 18, 2019
@brent-hoover
Merge pull request reactioncommerce#30 from sportsdirect/develop-test…
1669ce2 
…-int-deploy

Develop test int deploy
@Akarshit Akarshit mentioned this issue Jun 15, 2021
Merged
@snyk-bot snyk-bot mentioned this issue Jul 7, 2021
Open
@snyk-bot snyk-bot mentioned this issue Sep 22, 2021
Closed
This was referenced Mar 2, 2022
Closed
Closed
This was referenced Jul 20, 2022
Open
Open
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #30 from reactioncommerce/akarshit-fix-shop-types
001459f 
fix: type from input query saved
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #30 from reactioncommerce/dependabot/npm_and_yarn/…
27027c5 
…glob-parent-5.1.2

chore(deps): bump glob-parent from 5.1.1 to 5.1.2
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #30 from reactioncommerce/snyk-upgrade-14bcae7747b…
bbac792 
…90a314c5bfc8df18b062c

[Snyk] Upgrade @reactioncommerce/api-utils from 1.14.3 to 1.16.5
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #30 from reactioncommerce/dependabot/npm_and_yarn/…
d83f930 
…path-parse-1.0.7

chore(deps): Bump path-parse from 1.0.6 to 1.0.7
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #30 from reactioncommerce/dependabot/npm_and_yarn/…
97e69a4 
…path-parse-1.0.7

chore(deps): bump path-parse from 1.0.6 to 1.0.7
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@MohanNarayana
Merge pull request #30 from reactioncommerce/fix-28-mohan-multishop-d…
fe93538 
…up-tag-create

fix: allow different shops to have same tag name
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #30 from reactioncommerce/dependabot/npm_and_yarn/…
b8a72c5 
…path-parse-1.0.7

chore(deps): Bump path-parse from 1.0.6 to 1.0.7
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #30 from reactioncommerce/akarshit-chore-integrati…
92595d7 
…on-test

chore: add integration tests
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@MohanNarayana
Merge pull request #30 from reactioncommerce/fix-mohan-007-anonymousC…
cc80959 
…artByCartId

fix: find by _id OR anonymousAccessToken
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #30 from reactioncommerce/snyk-upgrade-917b121ba7d…
3b72071 
…82894ba35674907eef640

[Snyk] Upgrade @reactioncommerce/api-utils from 1.16.5 to 1.16.7
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #30 from reactioncommerce/snyk-upgrade-068b37392f5…
81db19d 
…9aaa46a1acc3012cb0091

[Snyk] Upgrade @reactioncommerce/api-utils from 1.16.7 to 1.16.9
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #30 from reactioncommerce/dependabot/npm_and_yarn/…
fd7af5d 
…validator-13.7.0

chore(deps): Bump validator from 13.6.0 to 13.7.0
brent-hoover pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #30 from reactioncommerce/snyk-upgrade-d893ac3ba76…
43707f9 
…c9163ea6bada5b158311d

[Snyk] Upgrade @reactioncommerce/api-utils from 1.16.6 to 1.16.9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aaronjudd @bstocks