Page MenuHomePhabricator

Reword post edit notification to reflect save->publish change on desktop
Closed, ResolvedPublic1 Estimated Story Points

Description

I thought the word "saved" was replaced with "published" recently. I see most of the messages reflecting the change except the success message of VE. It still shows "Your edit was saved." and not "Your edit was published."

Reported: https://www.mediawiki.org/wiki/Topic:U4x93yxf8jom1pgc

Event Timeline

TheDJ triaged this task as Medium priority.Jan 2 2018, 10:03 AM
TheDJ created this task.
Deskana set the point value for this task to 1.Jan 2 2018, 5:37 PM
Deskana moved this task from To Triage to TR1: Releases on the VisualEditor board.

Here's another instance that probably has to be changed,

Screenshot_2018-01-03-23-56-21.png (960×540 px, 100 KB)

I guess it's related but different because the message shown is a little different

Change 348749 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] postEdit: Show a different success messsage on publish vs. save wikis

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

Change 348749 merged by jenkins-bot:
[mediawiki/core@master] postEdit: Show a different success messsage on publish vs. save wikis

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

GTrang subscribed.

When I save an edit on Wikipedia, it still says "Your edit was saved", not "Your edit was published". I am thus reopening this, which needs a more careful patch to be actually fixed.

Deskana subscribed.
In T183901#4030576, @GeoffreyT2000 wrote:

When I save an edit on Wikipedia, it still says "Your edit was saved", not "Your edit was published". I am thus reopening this, which needs a more careful patch to be actually fixed.

Changes to code do not take immediate effect. Tasks are closed as resolved when the code is merged and will be automatically deployed. The change will go live next week.

Please ignore my above comment. I completely misread the ticket.

@GeoffreyT2000 is right; this isn't working, and it should be. The below screenshot is from the English Wikipedia.

image.png (129×459 px, 13 KB)

Change 422174 had a related patch set uploaded (by DLynch; owner: DLynch):
[mediawiki/extensions/VisualEditor@master] DesktopArticleTarget: use -published message instead of -saved

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

DLynch moved this task from Incoming to Code review on the VisualEditor (Current work) board.
DLynch added subscribers: Jdforrester-WMF, DLynch.

This is the postedit-confirmation-saved message, which needs to get adjusted to postedit-confirmation-published in a few places.

Patch handles the VE cases.

There's also resources/src/mediawiki.action/mediawiki.action.view.postEdit.js which uses saved directly, if it's the value stored in the wgPostEdit config, which the classic EditPage setPostEditCookie does. Since that's entirely an old-editor path, I think, should it be updated as well or left alone?

Change 422174 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] DesktopArticleTarget: use -published message instead of -saved

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

I just noticed we haven't changed the wording of post edit notification for mobile VE to be "Published". Shouldn't we change it there as well?

Screen Shot 2018-03-27 at 3.23.49 PM.png (190×1 px, 19 KB)

I just noticed we haven't changed the wording of post edit notification for mobile VE to be "Published". Shouldn't we change it there as well?

Screen Shot 2018-03-27 at 3.23.49 PM.png (190×1 px, 19 KB)

@Ryasmeen You seem to be second person to report that on this task. T183901#3874213

I'm wondering this actually requires a separate task: T191104

Deskana renamed this task from Reword post edit notification to reflect save->publish change to Reword post edit notification to reflect save->publish change on desktop.Apr 3 2018, 6:51 PM
Deskana closed this task as Resolved.

This task was always about desktop because of the description, so this is resolved now.