Page MenuHomePhabricator

[M] Re-implement temp user created popup somewhere reusable
Closed, ResolvedPublic

Description

The temp user popup is implemented in mediawiki.action.view.postEdit.js, but is likely to be needed by more workflows than editing (see parent task).

We should make it a reusable component.

QA Results - DE Beta

Event Timeline

Tchanders renamed this task from Re-implement temp user created popup somewhere reusable to [M] Re-implement temp user created popup somewhere reusable.Oct 30 2023, 4:29 PM

Change 972437 had a related patch set uploaded (by Tchanders; author: Tchanders):

[mediawiki/core@master] Create module for showing a popup when a temporary user is created

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

Change 972437 merged by jenkins-bot:

[mediawiki/core@master] Create module for showing a popup when a temporary user is created

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

Testing notes

This shouldn't change the current functionality, and should only need regression testing.

Environment:

  • Just needs temp accounts enabled, so test on de beta or on local (with $wgAutoCreateTempUser['enabled'] = true)

Steps:

  1. Log out
  2. Make an edit to some article
  3. After saving the edit, the page should refresh and a popup should appear:

image.png (225×370 px, 24 KB)

@Tchanders Temp account pops up after you save an edit when you are logged out. The only thing is the wording is different from the screenshot above and what I got in de beta. Which is the correct one?

Status: ❓NMI
Environment: Beta: 1.42.0-alpha (3d1fbaa)
OS: macOS Sonoma 14.0
Browser: Chrome 119, Firefox 119, Safari 17.0
Skins. Vector 2022
Device: MBA M2
Emulated Device:: n/a
Test Links:
https://de.wikipedia.beta.wmflabs.org/w/index.php?title=Black_Ice_(Album)&stable=0&shownotice=1

❓AC1: https://phabricator.wikimedia.org/T349220

AboveLink- Wording is different
2023-11-16_16-28-40.png (173×314 px, 48 KB)
2023-11-16_15-21-52.png (147×237 px, 13 KB)

Good

2023-11-16_15-14-43.png (842×3 px, 228 KB)

@GMikesell-WMF I think the version you're seeing might be the browser's translation of the German message inside the popup. If I ask my browser to translate de.wikipedia.beta for me, I get the same message. But if I visit https://de.wikipedia.beta.wmflabs.org/wiki/MediaWiki:Postedit-temp-created/en I see what the actual English message would be.

@Tchanders Great, thanks for the information! Otherwise, I didn't come across anything else with the functionality not working as designed. I will move this to Done. Thanks for all your work!

Status: ✅ PASS
Environment: Beta: 1.42.0-alpha (2bb72a1)
OS: macOS Sonoma 14.0
Browser: Chrome 119
Skins. de beta
Device: MBA M2
Emulated Device:: n/a
Test Links:
https://de.wikipedia.beta.wmflabs.org/w/index.php?title=Black_Ice_(Album)&stable=0&shownotice=1

✅AC1: https://phabricator.wikimedia.org/T349220

2023-11-16_16-28-40.png (173×314 px, 48 KB)
2023-11-17_08-29-10.png (367×1 px, 78 KB)