-
Notifications
You must be signed in to change notification settings - Fork 32
🎨📝 web-api: New batch of improved user-facing messages #7944
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
🎨📝 web-api: New batch of improved user-facing messages #7944
Conversation
7aeae80 to
de6937b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7944 +/- ##
==========================================
- Coverage 87.84% 87.78% -0.06%
==========================================
Files 1849 1423 -426
Lines 71335 59152 -12183
Branches 1250 626 -624
==========================================
- Hits 62663 51926 -10737
+ Misses 8310 7016 -1294
+ Partials 362 210 -152
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
@mergify queue |
🛑 The pull request could not be mergedThis could be related to an activated branch protection or ruleset rule that prevents us from merging. (details: 2 of 6 required status checks are expected.) |
sanderegg
left a comment
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.
maybe search for "studies" as well
services/web/server/src/simcore_service_webserver/folders/_common/exceptions_handlers.py
Outdated
Show resolved
Hide resolved
services/web/server/src/simcore_service_webserver/projects/_controller/_rest_exceptions.py
Show resolved
Hide resolved
0f5a362 to
14b3668
Compare
…rity and consistency
d08b55b to
0e93266
Compare
|
matusdrobuliak66
left a comment
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.
Nice 👍
|
This pull request has been removed from the queue for the following reason: Pull request #7944 has been dequeued. The pull request could not be merged. This could be related to an activated branch protection or ruleset rule that prevents us from merging. (details: 2 of 6 required status checks are expected.). You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it. |



What do these changes do?
Introduces a new batch of user-facing messages, generated with the help of AI prompts and manually reviewed for accuracy.
Workflow
user_messagefunction.Related issue/s
How to test
It is important to audit the outputs
Dev-ops
None