Page MenuHomePhabricator
Create Task
Maniphest T95065

Email confirmation workflow doesn't work correctly from a Mobile device
Closed, ResolvedPublic2 Estimated Story Points
Actions

Description

There are a couple problems with the Email confirmation workflow from a mobile device. First, it doesn't tell you why you are being asked to log in (on desktop, it says "Please log in to confirm your email address." above the log in form). And then after you log in, it redirects you to the log in form again (even though you're already logged in).

Details

SubjectRepoBranchLines +/-
Rewrite mobile login page to be less hacky and closer to core versionmediawiki/extensions/MobileFrontendmaster+99 -210
Customize query in gerrit

Related Objects

Event Timeline

kaldari created this task.Apr 3 2015, 11:20 PM
kaldari raised the priority of this task from to Needs Triage.
kaldari updated the task description. (Show Details)
kaldari added projects: Web-Team-Backlog, MobileFrontend.
kaldari subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 3 2015, 11:20 PM
• bmansurov moved this task from Incoming to 2015-16 Q3 on the Web-Team-Backlog board.Apr 6 2015, 5:11 PM
• KLans_WMF moved this task from 2015-16 Q3 to Old Unorganized Resolved Tasks on the Web-Team-Backlog board.Apr 14 2015, 8:52 PM
• KLans_WMF added a project: Mobile-Web-Sprint-45-Snakes-On-A-Plane.
• KLans_WMF set Security to None.
kaldari renamed this task from Email confirmation workflow doesn't work correctly from a Mobile device to (2) Email confirmation workflow doesn't work correctly from a Mobile device.Apr 15 2015, 5:14 PM
phuedx renamed this task from (2) Email confirmation workflow doesn't work correctly from a Mobile device to Email confirmation workflow doesn't work correctly from a Mobile device.Apr 15 2015, 6:21 PM
phuedx edited a custom field.
phuedx added subscribers: Florian, phuedx.

@kaldari: It looks like @Florian is working on a related task: T95903

phuedx added a comment.Apr 17 2015, 10:40 AM

@kaldari: Did you start work on this?

kaldari moved this task from Needs Analysis to Doing on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 17 2015, 10:02 PM
kaldari added a comment.Apr 17 2015, 10:58 PM

The first part of this bug is caused by the fact that we purposefully throw away all warning messages on the mobile login screen :P

kaldari claimed this task.Apr 17 2015, 11:07 PM
gerritbot subscribed.Apr 18 2015, 1:02 AM

Change 204977 had a related patch set uploaded (by Kaldari):
WIP: Rewrite mobile login page to be less hacky and closer to core version

https://gerrit.wikimedia.org/r/204977

gerritbot added a project: Patch-For-Review.Apr 18 2015, 1:02 AM
phuedx moved this task from Doing to Code Review on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 21 2015, 2:30 PM
phuedx moved this task from Code Review to -1 (Needs More Work) on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 21 2015, 2:58 PM
phuedx moved this task from -1 (Needs More Work) to Code Review on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 22 2015, 7:44 AM
phuedx moved this task from Code Review to -1 (Needs More Work) on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 24 2015, 1:07 PM
phuedx moved this task from -1 (Needs More Work) to Code Review on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 24 2015, 7:11 PM
Jdlrobson added a project: Gather Sprint Forward.Apr 24 2015, 11:06 PM
Jdlrobson moved this task from In Analysis to Code review needed on the Gather Sprint Forward board.
gerritbot added a comment.Apr 27 2015, 9:16 AM

Change 204977 merged by jenkins-bot:
Rewrite mobile login page to be less hacky and closer to core version

https://gerrit.wikimedia.org/r/204977

Diffusion mentioned this in rMEXT6a4a56db2b90: Updated mediawiki/extensions Project: mediawiki/extensions/MobileFrontend….Apr 27 2015, 9:16 AM
phuedx moved this task from Code Review to Done on the Mobile-Web-Sprint-45-Snakes-On-A-Plane board.Apr 27 2015, 9:17 AM
Jdlrobson mentioned this in rEMFR04e869f12ff9: Rewrite mobile login page to be less hacky and closer to core version.Apr 27 2015, 9:17 AM
phuedx closed this task as Resolved.Apr 27 2015, 9:17 AM
• Jhernandez moved this task from Code review needed to Ready for signoff on the Gather Sprint Forward board.Apr 27 2015, 9:18 AM
Jdlrobson mentioned this in T97460: Update browser tests after login/sign up form warning changes.Apr 28 2015, 7:12 PM
Jdlrobson removed a project: Gather Sprint Forward.Apr 28 2015, 11:03 PM
• Phabricator_maintenance moved this task from Old Unorganized Resolved Tasks to 2014-15 Q4 on the Web-Team-Backlog board.Mar 7 2018, 1:29 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL