-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improvements to the InputEditor #462
Conversation
Fixing the failing tests. |
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.
Some small requests
.gitignore
Outdated
@@ -349,3 +349,6 @@ MigrationBackup/ | |||
# Ionide (cross platform F# VS Code tools) working folder | |||
.ionide/ | |||
/COMETwebapp/Tools | |||
|
|||
# Jetbrains folders | |||
.idea/ |
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.
Please add a new line at the end of the file
…ext fixtures to match the new business rules and changes made.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## development #462 +/- ##
===============================================
- Coverage 78.38% 78.31% -0.07%
===============================================
Files 328 330 +2
Lines 7513 7600 +87
Branches 760 772 +12
===============================================
+ Hits 5889 5952 +63
- Misses 1393 1414 +21
- Partials 231 234 +3
☔ View full report in Codecov by Sentry. |
… the project's convention.
Working on the coverage, just a bit more 😉 |
Kudos, SonarCloud Quality Gate passed! 0 Bugs 75.8% Coverage The version of Java (11.0.20.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. |
Prerequisites
Description
The following changes will allow us to show/hide the Name and Short name fields on the InputEditor, via a configuration file. Additionally, a MultiComboBox component was created as DevExpress, by default, does not have a dropdown component which enables multiple choice.