-
Notifications
You must be signed in to change notification settings - Fork 8
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
548115 - [Passage] provide ability to create product version from issue license wizard : part 2 #184
Conversation
Use proper type names Signed-off-by: Alexander Fedorov <[email protected]>
Extract facilities to get localized type names Signed-off-by: Alexander Fedorov <[email protected]>
Shift nested wizard window for a while Signed-off-by: Alexander Fedorov <[email protected]>
Rework LocWorkbench#createDomainResource method Signed-off-by: Alexander Fedorov <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #184 +/- ##
===========================================
- Coverage 9.18% 9.08% -0.11%
Complexity 105 105
===========================================
Files 503 516 +13
Lines 13831 13989 +158
Branches 1126 1135 +9
===========================================
Hits 1271 1271
- Misses 12525 12683 +158
Partials 35 35
Continue to review full report at Codecov.
|
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.
One question and two stream style notes, no changes are mandatorily required.
....workbench/src/org/eclipse/passage/loc/internal/workbench/wizards/InnerClassifierWizard.java
Outdated
Show resolved
Hide resolved
...kbench/src/org/eclipse/passage/loc/internal/workbench/wizards/InnerClassifierWizardPage.java
Outdated
Show resolved
Hide resolved
...ard.ui/src/org/eclipse/passage/moveto/lic/internal/features/model/FeaturesClassMetadata.java
Show resolved
Hide resolved
Add tests for EClassResources Add tests for EClassName Signed-off-by: Alexander Fedorov <[email protected]>
Changed to method references Signed-off-by: Alexander Fedorov <[email protected]>
Replace with ClassifierInitializer#newObjectName with EObjectDefaultName Signed-off-by: Alexander Fedorov <[email protected]>
Replace with ClassifierInitializer#newObjectIdentifier with EObjectDefaultIdentifier Signed-off-by: Alexander Fedorov <[email protected]>
@eparovyshnaya please have a look, I fixed all the labels for nested wizards |
Fixed several annoying issues for "select or create" scenario: