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
feat(MyProjectsUI):Add additional filter to My Projects homepage based on project clearing state. #1137
feat(MyProjectsUI):Add additional filter to My Projects homepage based on project clearing state. #1137
Conversation
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.
Tested it working fine. Except some minor review comments.
...w360-portlet/src/main/java/org/eclipse/sw360/portal/portlets/homepage/MyProjectsPortlet.java
Outdated
Show resolved
Hide resolved
@@ -66,7 +66,25 @@ | |||
<core_rt:if test="${userRoles==null||userRoles.SECURITY_RESPONSIBLES}">checked="checked"</core_rt:if>> | |||
<label class="form-check-label" for="securityResponsible">Security Responsible</label> | |||
</li> | |||
<li><hr class="my-2" /></li> | |||
<li class="dropdown-header">Clearing State</li> |
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.
It would be good to read the labels from i18N properties.
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.
Followed the existing approach,which is working in production for roles label.
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.
hm, it was for internationalization. But it's ok for now.
...w360-portlet/src/main/java/org/eclipse/sw360/portal/portlets/homepage/MyProjectsPortlet.java
Outdated
Show resolved
Hide resolved
...w360-portlet/src/main/java/org/eclipse/sw360/portal/portlets/homepage/MyProjectsPortlet.java
Outdated
Show resolved
Hide resolved
6188c6a
to
68fbfe3
Compare
…d on clearing state. Signed-off-by: ravi110336 <kumar.ravindra@siemens.com>
68fbfe3
to
0b7818d
Compare
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.
Code looks good. Tested again successfully.
Issue:
Suggest Reviewer
How To Test?
Checklist
Must:
Signed-off-by: Ravindra Kumar kumar.ravindra@siemens.com