Page MenuHomePhabricator

Section publishing on desktop doesn't publish to sandbox when it should
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • 1. Open the Special:ContentTranslation page with the unified-dashboard URL parameter set to true, on a desktop device.
  • 2. Select a section suggestion from the dashboard list and pick a section to translate. The desktop editor with the section translation loaded should be displayed.
  • 3. Complete the section translation, select as publishing target the user sandbox option and publish your section translation.

What happens?:
The section is being published into the main namespace.

What should have happened instead?:
The section should have been published into the user sandbox.

Other information (browser name/version, screenshots, etc.):
Screencast of the issue on greek (el) production wiki:

Event Timeline

ngkountas changed the task status from Open to In Progress.Nov 15 2023, 3:19 PM
ngkountas claimed this task.
ngkountas triaged this task as High priority.
ngkountas updated the task description. (Show Details)
ngkountas added a project: SectionTranslation.

Change 975053 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] CX: Support section publishing to sandbox

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

Change 975053 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX: Support section publishing to sandbox

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

Test status: QA PASS

  • This now saves to namespace as it is selected.
  • Also the namespace selection persists

@EChukwukere-WMF The above screencast refers to an article translation using Content Translation. This task is about section translation using Content Translation, which is a different case. Moving this back to "Needs QA" column.

Moving to Pending input for QA column as discussion needed with @ngkountas

This task has not been fixed. Moving this to backlog. I will also submit a screencast of the still existing issue, soon.

Change #1014507 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] CX: Set the "publishNamespace" property when namespace is changed

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

Change #1014508 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] SX publishing on desktop: Use returned target title for success message

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

Change #1014507 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX: Set the "publishNamespace" property when namespace is changed

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

Change #1014508 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] SX publishing on desktop: Use returned target title for success message

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