Page MenuHomePhabricator

Create IP /Temp account sheet component, present from Article
Closed, ResolvedPublic3 Estimated Story Points

Description

  • When user is about to enter the article editor, get their auth status.
  • Create a new sheet component. Allow it to be configured with image and copy for both IP users ("You are not logged in") and temporary account users ("You are using a temporary account".). See the "You are not logged in" and "You are using a temporary account" UI modal component in T337250 for reference.
  • Present modal component created in step 2 before they enter the editor, IF their auth state from step 1 indicates ip or temporary. Configure it to IP or Temporary state.
  • The area that presents the editor is here - ArticleViewController's presentEditor method. We can try to modify this method to present the new component.

Test notes:
Either using IP or Temporary Accounts, or switching the statement
Navigate to an article
Press edit
Ensure pop up shows up, repeat with other type of account

Experimental Build 238

Event Timeline

Tsevener renamed this task from [M] Create Temp account sheet component to [M] Create IP /Temp account sheet component.
Seddon triaged this task as Low priority.Sep 6 2024, 2:08 PM
Seddon moved this task from Needs Triage to Up next on the Wikipedia-iOS-App board.
HNordeenWMF raised the priority of this task from Low to Medium.Dec 19 2024, 10:23 PM
Seddon set the point value for this task to 3.Jan 8 2025, 7:03 PM
HNordeenWMF lowered the priority of this task from Medium to Low.Jan 24 2025, 6:01 PM
Seddon renamed this task from [M] Create IP /Temp account sheet component to Create IP /Temp account sheet component.Jan 27 2025, 1:35 PM

Update from Robin: change X buttons -> Done in upper lefthand corner to match

Tsevener renamed this task from Create IP /Temp account sheet component to Create IP /Temp account sheet component, present from Article.Feb 5 2025, 2:24 PM
Tsevener updated the task description. (Show Details)

This is ready to QA. Moving to Blocked / Waiting since it's hanging out in a feature branch (temp-accounts) until all temp accounts work is done.

ABorbaWMF subscribed.

Testing on 7.7.3.236

When logged out and attempting to edit, I am seeing the temp sheet, but tapping continue without logging in does nothing.

Looks good to me!

The screen presented on first tap into editor is correct. The one presented from article view (red icon in top left) matches that one. According to the designs it should be slightly different, but it has the same functionality, so it's fine by me that they're the same (and @GOlson-WMF I feel like we agreed on this a while back!)

I’m never seeing the illustration for this screen in all the places it’s used ( Left is design, right is Beta) - could that be added in (in all places where the "You are using a temporary account" screen is used?

image (19).png (375×812 px, 33 KB)
IMG_0285 (1).PNG (1,179×2,556 px, 307 KB)