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

Add fields required for reply request reason & voting reason #13

Merged
merged 1 commit into from Mar 11, 2018

Conversation

@MrOrz
Copy link
Member

commented Feb 24, 2018

In order to implement cofacts/rumors-api#67 , we should add these fields to DB.

Existing reply request does not have reason fields. They can't be given feedbacks anyway, thus I think we don't need to migrate them.

@MrOrz MrOrz self-assigned this Feb 24, 2018

@MrOrz MrOrz requested review from godgunman, darkbtf and GoreStarry Feb 24, 2018

@MrOrz

This comment has been minimized.

Copy link
Member Author

commented Mar 11, 2018

Not breaking things but blocks implementation, thus merge first

@MrOrz MrOrz merged commit 109776e into master Mar 11, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@MrOrz MrOrz deleted the create-article-reason branch May 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.