-
Notifications
You must be signed in to change notification settings - Fork 4
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
State Machine #6
Comments
@dronov @lucasocon I am thinking we can Install wordpress to provide the blogging solution. |
Okay @samuels410 no problem. |
@samuels410 👍 |
@dronov Please do that. |
@samuels410 ok, I will make it first. Please comment another issue #5 |
Blog post moved to wordpress. |
Hi. I think I found a better solution for updating state of blog posts.
No need to keep actions 'publish, undo_publish' in controllers like here. There is a better solution to add State Machine to Post model for keeping the right business logic. In my opinion controllers should'not make actions like these and the Post model will be with more clearer code.
If you're agreed I can fix it later after #5
The text was updated successfully, but these errors were encountered: