Skip to content
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

Use same method everywhere to decide if an element is visible in the front #1355

Open
amiedes opened this issue Jan 19, 2018 · 0 comments
Open
Labels

Comments

@amiedes
Copy link
Contributor

amiedes commented Jan 19, 2018

Originally this was done with a .published? method / scope. But logic is becoming more complicated and now this is not sufficient condition in some cases, like with published person events when the person is drafted, published pages on a process when the process is drafted etc. drafted != visible in the frontend anymore so I propose to use everywhere a public? method or public scope to achieve this.

@amiedes amiedes changed the title Use same method everywhere to decide if a page is visible in the front Use same method everywhere to decide if an element is visible in the front Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant