-
Notifications
You must be signed in to change notification settings - Fork 32
♻️ refactor project listing DB function (🚨 We no longer list projects that do not have a product assigned) #6692
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
♻️ refactor project listing DB function (🚨 We no longer list projects that do not have a product assigned) #6692
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6692 +/- ##
==========================================
+ Coverage 87.95% 88.88% +0.93%
==========================================
Files 1569 1272 -297
Lines 63117 54419 -8698
Branches 2106 897 -1209
==========================================
- Hits 55512 48371 -7141
+ Misses 7280 5914 -1366
+ Partials 325 134 -191
Continue to review full report in Codecov by Sentry.
|
bisgaard-itis
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.
Cool! Thanks a lot for the effort 😄
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.
aren't we missing some tests in here? not sure I get the gist of the scopes.
services/web/server/src/simcore_service_webserver/projects/db.py
Outdated
Show resolved
Hide resolved
|
We actually have a lot of listing tests. If they pass, it means nothing was broken. As for the scopes, the idea is to differentiate the type of filtering to apply. For folders, this could mean:
The same logic applies to workspaces. In the context of workspaces, the types are:
Does this make more sense? We can discuss further offline if needed. |
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.
all clear, thanks for the explanations
|
ce9d3a5
into
ITISFoundation:master



What do these changes do?
Related issue/s
How to test
Dev-ops checklist