Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Flagged Reason migrate fix#3717

Merged
aaronschachter merged 3 commits intoDoSomethingArchive:devfrom
aaronschachter:flagged_reason
Jan 13, 2015
Merged

Flagged Reason migrate fix#3717
aaronschachter merged 3 commits intoDoSomethingArchive:devfrom
aaronschachter:flagged_reason

Conversation

@aaronschachter
Copy link
Contributor

Closes #3652

Fixes error in dosomething_reportback_update_7018, which queried by the incorrect column when gathering the Flagged reasons. This is a great example of why it makes more sense to simplify the build and store flaggings as a column on the reportback itself.

Also exposes Review Source when viewing the Reportback File entity, to better spot check migrated Flagging records.

@sergiitk
Copy link
Contributor

👍

aaronschachter added a commit that referenced this pull request Jan 13, 2015
@aaronschachter aaronschachter merged commit 2ab8261 into DoSomethingArchive:dev Jan 13, 2015
@aaronschachter aaronschachter deleted the flagged_reason branch January 13, 2015 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Refactor Flagged Reportbacks

2 participants