Page MenuHomePhabricator

OAuth MediaWiki extension post-submission message is confusing after updating an app's grants
Closed, DeclinedPublic

Description

After successfully updating grants to an app (at [[testwiki:Special:MWOAuthManageMyGrants/manage/72]]), I get the following message:

Manage account OAuth grants
   Navigation: [Accepted consumer list]

The access token for this consumer has been updated.

Return to Main Page.

There should be a link back to [[testwiki:Special:MWOAuthManageMyGrants/manage/72]]. And I'm not sure "Return to Main Page" is needed at all.

I'm also not sure this post-submission confirmation page is needed. Perhaps it could just reload [[testwiki:Special:MWOAuthManageMyGrants/manage/72]] and output a success message (green checkmark, etc.).

Or perhaps it update without even requiring a reload/refresh.


Version: unspecified
Severity: normal

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:35 AM
bzimport set Reference to bz55721.
bzimport added a subscriber: Unknown Object (MLST).

Doesn't block deployment. However, does need to be fixed at a higher priority, as it's very confusing.

I'd like to see this changed to "Your preferences for this application have been updated.", with a link to return to preferences instead of the main page.

Tgr set Security to None.

There should be a link back to [[testwiki:Special:MWOAuthManageMyGrants/manage/72]]

I'm not sure that's useful. I just changed my preferences; I probably don't want to go back and change them again. (And if I really want to, the browser's back button still works.)

Change 222239 had a related patch set uploaded (by Gergő Tisza):
Improve management IF return messages

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

Change 222239 merged by jenkins-bot:
Improve management IF return messages

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

As said above, I don't think it makes sense to send the user to update the consumer they just updated. (The main page link is now removed and after management events which have an associated queue (e.g. accepting a proposed consumer <-> proposed queue), there is a link back to the queue.)