[IS-11] Change fail message to list ids as mentionned in a few comments #793
Comments
|
A general check of messages would be nice as well, variously I noticed "IS-11", "IS11" and "IS_11" for example, or a copy/pasted error message that hadn't been updated for its new location. |
gwgeorgea
added a commit
to gwgeorgea/nmos-testing
that referenced
this issue
Apr 28, 2023
…return either the pertinent IDs or make the message clearer. All messages were kind of normalized. AMWA-TV#793
gwgeorgea
added a commit
to gwgeorgea/nmos-testing
that referenced
this issue
Apr 28, 2023
…return either the pertinent IDs or make the message clearer. All messages were kind of normalized. AMWA-TV#793
Merged
|
@garethsb I have not found any "IS11" and "IS_11", could you flag them for me in this review if you do find them. |
|
@gwgeorgea forgive me, I think you fixed them all already, thanks 🙏 |
|
Resolved by #802. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In PR-761, there is quite a lot of messages that would benefit from list the id of the problematic ressource.
This is mentionned in a few comments here, but there is much more to change.
#761 (comment)
#761 (comment)
The text was updated successfully, but these errors were encountered: