Differential D11289

T6870, Return to previous page after login via header
ClosedPublic
Actions

Authored by lpriestley on Jan 8 2015, 11:23 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T6870: Return to previous page after login via header

Commits

Restricted Diffusion Commit
rP6c7e0e280713: T6870, Return to previous page after login via header

Summary

Fixes T6870, logging in from a public object should land on that object.

Test Plan

Navigate to a maniphest task in a logged out state, login, landing page should be maniphest task.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 27122.Jan 8 2015, 11:23 PM

lpriestley retitled this revision from to T6870, Return to previous page after login via header.

lpriestley updated this object.

lpriestley edited the test plan for this revision. (Show Details)

lpriestley added a reviewer: epriestley.

lpriestley added a task: T6870: Return to previous page after login via header.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 8 2015, 11:23 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Jan 8 2015, 11:56 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jan 8 2015, 11:56 PM

epriestley awarded a token.Jan 8 2015, 11:56 PM

Closed by commit rP6c7e0e280713: T6870, Return to previous page after login via header (authored by lpriestley, committed by epriestley). · Explain WhyJan 8 2015, 11:56 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

application/

PhabricatorAuthApplication.php

7 lines

Diff 27124

View Options

src/applications/auth/application/PhabricatorAuthApplication.php

T6870, Return to previous page after login via headerClosedPublicActions