Last modified: 2013-04-22 16:16:41 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T39468, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 37468 - Add step-backwards from "thanks" with file errors to "describe" to allow changes if necessary
Add step-backwards from "thanks" with file errors to "describe" to allow chan...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 37462
  Show dependency treegraph
 
Reported: 2012-06-11 19:59 UTC by Mark Holmquist
Modified: 2013-04-22 16:16 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Mark Holmquist 2012-06-11 19:59:37 UTC
Rather than just stopping and offering "retry" as the only option, errors on the final step should probably give the user the option to go backwards and add in the right filename, change the description, and so on. This is necessary especially because we may soon be finishing the description, etc., _before_ the stash upload finishes, so any errors from both upload attempts would need to be shown on the description page. The fix for bug 32247 already allows removal of the file from "describe" (so if it's a duplicate error, the file can be removed easily), so any error could feasibly be fixed, or at least worked around, with the addition of this backwards mobility.

Note that I still don't think backwards navigation makes sense in any other case, because other data would not need to be changed based on the server's input.
Comment 1 Mark Holmquist 2012-06-11 20:30:45 UTC
OK, as it turns out, this capability exists! The only thing was, for unknown and duplicate errors, we weren't calling the right methods.

Patch. Patch need review. https://gerrit.wikimedia.org/r/10995
Comment 2 Mark Holmquist 2012-08-14 16:59:44 UTC
Correction: Patch. Patch in Gerrit.
Comment 3 Robin Pepermans (SPQRobin) 2012-08-30 02:52:20 UTC
As the referenced gerrit patch is merged, I assume this bug can be closed?

Also, this bug seems to be about the same or at last very related to bug 29594, but well.
Comment 4 Alex Monk 2013-01-08 00:54:00 UTC
So is this done now...?
Comment 5 Andre Klapper 2013-01-21 09:01:26 UTC
Twice no feedback => Assuming this is FIXED as per merged Gerrit patch.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links