-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Change error message in editor #3629
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3629 +/- ##
===========================================
- Coverage 44.85% 44.84% -0.01%
===========================================
Files 257 257
Lines 19871 19873 +2
Branches 3131 3132 +1
===========================================
Hits 8913 8913
- Misses 10958 10960 +2
Continue to review full report at Codecov.
|
'Please add feedback for the user if they are to return to the ' + | ||
'same state again.') | ||
'Please add feedback for the user in the [All other answers] ' + | ||
'rule if they are to return to the same state again.') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest removing "if they are to return to the same state again" as currently it sounds like a conditional - i.e. the editor has to determine whether the user would return to the same state again, and if so fix things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed it, please take a look again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
This changes the error message in the editor to clarify where the author should add feedback in the case where the default rule has no feedback.