search

cassproject / cass-editor

View, create, edit, modify frameworks using an easy to use, embeddable iframe.
https://cassproject.github.io/cass-editor/
Apache License 2.0
3 stars 11 forks source link

Unable to create new ProgressionModel, as the editor doesn't recognize that description has been entered (or publication status has been selected) #1355

Closed mparsons-ce closed 4 weeks ago

mparsons-ce commented 2 months ago

@Lomilar SANDBOX I created a new progression model. Clicked Done. Then got error that description is required, but it exists image

I updated the description and tried again. Now I get an error that language is required, but exists. image

I also got error that Publication Status was required, but it is already selected. image

On production a similar error will occur for description. I found that I could just update the latter and then save successfully.

Lomilar commented 2 months ago

I was able to create a progression model by selecting a language 'English' vs 'en-us'. There was definitely some weird validation errors going on that required some clicking around. @gloverkari fysa

mparsons-ce commented 2 months ago

I tried creating a PM (in the sandbox) using English and still get an error. I then changed it to en, and clicked Done. Then the description was flagged as required (even though it had data). I did a simple update to description, and click Done. Then the language required message appeared again.

image

Lomilar commented 2 months ago

This can still occur, but sufficient clicking on and off the required fields can clear the issue.

Lomilar commented 2 months ago

hint: Blur validation isn't fully working, may want some sort of timeout based validation.

winonadeb commented 2 months ago

Since there is a workaround (click off of the field you edited before saving), is this still a blocker?

sedula27 commented 1 month ago

Issue with the required fields(Description/Publication Status) highlighted even though they are filled, this is not a consistent issue

sedula27 commented 4 weeks ago

https://github.com/cassproject/cass-editor/issues/1356 and this have the same issue with Publication Status and we have a temporary workaround

sedula27 commented 4 weeks ago

Refer #1356, they point to the same issue so closing this one