-
-
Notifications
You must be signed in to change notification settings - Fork 647
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
Decline articles #716
Decline articles #716
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.
Hi @tomhatzer, thank you very much for the contribution.
This mostly looks great, but I don't think we need the ability to undecline an article. Once it has been declined, it's typically because we don't fing the content to be appropriate.
Would it be possible to remove this functionality and I'll then review again. Thanks!
Hey @joedixon , I've just removed the undecline functionality. Looking forward to your review 😃 Thank you! |
database/migrations/2021_10_31_143501_add_declined_at_column_to_articles_table.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Dries Vints <dries@vints.io>
…rning article in DeclineArticle handle method
Thanks for the review @driesvints, all changes have been implemented. |
database/migrations/2021_10_31_143501_add_declined_at_column_to_articles_table.php
Show resolved
Hide resolved
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.
Looks good to me. I've been looking forward to this being built so thank you @tomhatzer
Thanks @tomhatzer! |
Hey!
This fixes #580 and adds decline and undecline functionality to the articles. It adds a new nullable column to the articles table named
declined_at
.Feel free to comment if you have any recommendations or if there's something missing.
Have a great day! 😃