Issue 2506893003: [Findit] After login, redirect to the targeted page for GETs, but original page for others like POS…

Issue 2506893003: [Findit] After login, redirect to the targeted page for GETs, but original page for others like POS… (Closed)

Created:
4 years, 1 month ago by stgao

Modified:
4 years, 1 month ago

Reviewers:
chanli, chanli1

CC:
chromium-reviews, infra-reviews+infra_chromium.org, lijeffrey

Target Ref:
refs/heads/master

Project:
infra

Visibility:
Public.

[Findit] After login, redirect to the targeted page for GETs, but original page for others like POSTs. BUG=665207 Committed: https://chromium.googlesource.com/infra/infra/+/f30f1fb806242e021e6ec66b5a9e9a7f729f24bd

Created: 4 years, 1 month ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+30 lines, -3 lines)			Patch
	M	appengine/findit/common/base_handler.py	View	1	1 chunk	+7 lines, -1 line	0 comments	Download
	M	appengine/findit/common/test/base_handler_test.py	View		1 chunk	+23 lines, -2 lines	0 comments	Download

Total messages: 7 (3 generated)

chanli1

On 2016/11/16 23:49:19, stgao (slow on Monday) wrote: > ptal lgtm

4 years, 1 month ago (2016-11-17 01:33:58 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2506893003/20001

4 years, 1 month ago (2016-11-17 02:11:53 UTC) #5

commit-bot: I haz the power

4 years, 1 month ago (2016-11-17 02:27:32 UTC) #7

Message was sent while issue was closed.

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/f30f1fb806242e021e6ec66b5a9e9...