Skip to content

Java: Add viewmodels [STUD-77111]#551

Open
viogroza wants to merge 3 commits into
developfrom
fix/java_viewmodels
Open

Java: Add viewmodels [STUD-77111]#551
viogroza wants to merge 3 commits into
developfrom
fix/java_viewmodels

Conversation

@viogroza
Copy link
Copy Markdown
Collaborator

@viogroza viogroza commented May 14, 2026

Add viewmodels for java package
Make java package and activities available also on cross-platform
Fix Params and dynamic param being mixed up
Add toggle for the user to be able to select between static and dynamic params

https://uipath.atlassian.net/browse/STUD-77111

Add viewmodels for java package
@viogroza viogroza force-pushed the fix/java_viewmodels branch from db6f9e7 to 247b2dd Compare May 15, 2026 12:37
@viogroza viogroza changed the title WIP Java: Add viewmodels [STUD-77111] Java: Add viewmodels [STUD-77111] May 18, 2026
Copy link
Copy Markdown
Collaborator

@sorinconstantin sorinconstantin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix the Sonar report on JavaScopeViewModel.cs

Toggle between static and dynamic properties
Fix improper usage of static params (even when dynamic was selected)
@viogroza
Copy link
Copy Markdown
Collaborator Author

@sorinconstantin the finding is valid, but I added a comment on the constructor why we need to keep that field (basically is a workaround to make sure that the viewmodels are properly loaded by studio)

@viogroza viogroza requested a review from sorinconstantin May 18, 2026 14:19
Address extra review findings
@viogroza viogroza force-pushed the fix/java_viewmodels branch from b84cdb4 to e6aa9b3 Compare May 18, 2026 15:06
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
10.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants