Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

point user review dashboard to ~new place #3546

Merged
merged 3 commits into from
Jun 11, 2015
Merged

Conversation

chadwhitacre
Copy link
Contributor

Fixes #3496.

@chadwhitacre
Copy link
Contributor Author

Ping @rohitpaulk. I'll leave this for a bit in case you're around. Since it's small, obvious, not user-facing, and useful for #3540, I'll probably self-merge before proceeding with #3540.

@rohitpaulk
Copy link
Contributor

We were allowing GET requests to toggle is_suspicious? Let me just fix that quick..

@chadwhitacre chadwhitacre mentioned this pull request Jun 11, 2015
@chadwhitacre
Copy link
Contributor Author

!m @rohitpaulk

@rohitpaulk
Copy link
Contributor

Hmm, wonder how 16b029d passed on Travis.

@rohitpaulk
Copy link
Contributor

Maybe the job started late and pulled the latest commit instead?

@rohitpaulk
Copy link
Contributor

Anyways. Does fail on my local, so merging.

rohitpaulk added a commit that referenced this pull request Jun 11, 2015
point user review dashboard to ~new place
@rohitpaulk rohitpaulk merged commit a3d4346 into master Jun 11, 2015
@rohitpaulk rohitpaulk deleted the fix-user-dashboard branch June 11, 2015 20:46
@chadwhitacre
Copy link
Contributor Author

Does fail on my local, so merging.

s/Does/Doesn't?

@chadwhitacre
Copy link
Contributor Author

;-)

@rohitpaulk
Copy link
Contributor

The first commit that should've failed does fail, @whit537 :P

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants