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
LPS-76354 #53599
LPS-76354 #53599
Conversation
…s behavior in the back end instead of relying on a disabled checkbox
Comment ci:test to trigger CI tests. |
/cc @jonathanmccann |
@brianchandotcom the initial comment is from Jonathan's original pull. |
ci:test |
Pull request test invoked at http://test-1-13.liferay.com/job/test-portal-acceptance-pullrequest(master). |
ci:test |
Pull request test invoked at http://test-1-10.liferay.com/job/test-portal-acceptance-pullrequest(master). |
Merged. Thank you. |
@drewbrokke Sending this through you since it seems more related to Users than Announcements, but let me know if you'd like to have this go elsewhere.
Based off of LPS-216, it seems the Website option should always be checked, however, we're depending on disabled checkboxes to achieve that. Instead, we should enforce this rule in the backend so that it never changes.
I would have liked to remove the entire Website option since I don't see the use for it but that requires a major version increase to portal-kernel which I'd like to avoid.
Let me know if you have any questions.