-
Notifications
You must be signed in to change notification settings - Fork 21
[O2B-1240] Display only physical productions by default and adds filtering for 'test' and 'debug' productions display #1940
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
[O2B-1240] Display only physical productions by default and adds filtering for 'test' and 'debug' productions display #1940
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1940 +/- ##
==========================================
+ Coverage 45.54% 45.77% +0.22%
==========================================
Files 1010 1014 +4
Lines 16680 16699 +19
Branches 2997 3006 +9
==========================================
+ Hits 7597 7644 +47
+ Misses 9083 9055 -28 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
lib/public/views/DataPasses/ActiveColumns/dataPassesActiveColumns.js
Outdated
Show resolved
Hide resolved
graduta
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.
I believe the addition of [include][byName] it adds complexity without any benefit. It is not the same as the numerical values where we have to specify operator and limit.
In this case, it is a simple inclusion of strings
I have a JIRA ticket
Notable changes for users:
Notable changes for developers:
Changes made to the database: