-
Notifications
You must be signed in to change notification settings - Fork 2
Commerce Datattype Price isn't saving in splitview #695
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
Comments
Can you upgrade to 13.2.1 and see if the issue is still present. Not sure there is anything in the release notes that suggest it would be fixed, but we'd need you to be on the latest v13 anyway before we can look into it. |
@mattbrailsford Hi Matt, |
@cryothic thanks for checking. I'll get this added to our backlog to investigate 👍 |
Ok, I've managed to replicate this. Ultimately the Price property waits for a "formSubmitting" event to write it's value back to the model, but it looks like both views trigger that event and so both views cause a save, but the second views wins and so because the value isn't synchronizing, the new value gets overwritten by the old value |
Ok, I've pushed a 13.2.2--preview.2 build to our nightly feed, if you'd like to give that a try I think it should solve your issue. |
Hi @mattbrailsford This seems to work. If I now change the price, it get's updated after saving in both views. I've noticed the field isn't syncing realtime though. But the saving self seems to work. |
Ahh, yea, good point. Just pushed a new 13.2.2--preview.3 build that should take care of that 👍 |
That was quick. |
Ok great, I'll schedule this for release next week but you are welcome to keep using the nightly in the meantime. |
|
Which component is this issue related to?
Umbraco Commerce (Core)
Which Umbraco Commerce version are you using? (Please write the exact version, example: 10.1.0)
13.1.4 (and still remains after updating to 13.2.1)
Bug summary
It seems the price field isn't saving when you have a page open in split-view.
I have this same issue in Vendr sites too, but I was hoping it was fixed in Commerce.
I have also tried saving field of type
Umbraco Commerce: Stock
andUmbraco Commerce: Measurements
, but they are saved correctly. The only problem I've found so far was with the price field.Specifics
No response
Steps to reproduce
Umbraco Commerce: Price
field.Expected result / actual result
I would expect after saving, the new values are in place.
But the price value isn't saved.
If I started with an empty product page, after saving the textfield contains text and the price field is empty.
If I started with an allready filled product page, the textfield gets updated but the price value gets reset to it's previous value.
pricefield_save.mp4
Dependencies
No response
This item has been added to our backlog AB#52353
The text was updated successfully, but these errors were encountered: