@Les4353 I'll be around next week, so feel free to tag me if you need help from someone familiar with the work while Anne's out ☺️

Steph to sign off

Working on ticket before it has points assigned big no no

Didn't get a chance to make this ticket, but adding it to the sprint anyways 🙃

Think I'm supposed to set it back to you?

I'm gonna be honest I have no idea how to read the heatmap chart, but I can confirm these metrics were added and the values seem reasonable. Thank you for doing this work!!

Left some comments on the user flow, but things are looking good - thank you so much for putting this all together @jwang!!

I know at least this one belongs

Thank you!!

Already slightly in progress

From sprint planning: @Sneha says no worries if all items empty state is not feature flagged and goes out in general

I'm thinking ReadingListsCustomLists, but if you wanna bikeshed on the name go ahead

Low/Medium priority for now as this is not planned for phase 1

Note - I feel like these have been sufficiently discussed with the team if you want to move this straight to sign off for the sake of the sprint. That said, I see we have more than only our team on this ticket, so if we want to keep this one around for centralizing discussion around the bigger picture I wouldn't object to that either (and tagging @HFan-WMF for his input as well)

Summary:

• as discussed, sidebar exploration will be deferred until a later date
• similarly, infochip interactivity is a bit underexplored from a codex perspective, so for now we're going to use toggle button and select (with an assumption that this can and will change down the line)
• all other tickets have been or will be created under the parent epic

Medium priority because it will need to get done eventually, but is descoped for the current phase of development

Note that on mobile it would be <Custom List Title> and on desktop it would be Saved / <Custom List Title>

Confirmed 0 rows remaining

toyofuku@stat1008:~$ analytics-mysql arwiki
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

I have the perms and can QA this

Yay I finally have time

Hello @Lock46! I'm going to try and get someone from the iOS team to take a look at this, but in the meantime - is there any chance this has to do with you being logged out? It's odd that one single list would be removed, but we'll definitely look into it

Nvm I was able to get to it

@aude if I don't get to this first - can you add QA steps for how someone who isn't either of us could make sure this is done (I recommended QAing via ensuring the DB errors drop off)

gr8 news - the sidebar menus stuff has been deprioritized in the initial version, so we can defer worrying about that until I have a chance to pick individuals' brains on the matter

See also T409354

This and T393977 are cousins in my mind

Low to medium priority as it's not currently a blocker for this project (low within the context of the epic, medium otherwise imo)

Explicitly, the scope of this work is principally anything that could be considered part of that bottom drawer component - no need to implement things like the pop up confirmation modal provided your solution doesn't make it actively hard to incorporate after the fact

Steph to review

Okay - I tend to hold onto tickets I worked on until they're in an easily reviewable and finalized state, but if you'd prefer I'm happy to move it

Not ready yet not passing tests

@Sneha if you don't mind taking a look at the patchdemo ^ both logged in and logged out to make sure the positioning/layout looks right, I had to interpret the figma a bit since the bookmark icon doesn't seem to be in exactly the specified position

Created T427942 (with a reference to T427943), so I think the follow up tasks are handled

This is ready for review - I seem to have run into the perennial issue of tests depending on a separate extension only being skippable, which more or less defeats the purpose of tests, but at least now the code is mergeable 🙃

Copying priority from T424089 that this is a spin-off of

Love the purple 💜

Jon to code review

😶😶😶

In progress although semi-stalled on discussing approach in fundraising x product collab channel

Survey enabled and confirmed it's live - noting here as well that it seems like (Spanish) translations still aren't in

I have thoughts/feelings/opinions about this task, so would love to call dibs if that's not too controversial

Grabbing this as discussed in eng enclave - I'm not positive it'll be done before the end of the sprint, but I'll do what I can

DMed @jwang to make sure I understood the instrumentation spec correct, but assuming I did so moving to code review - if something needs to be corrected, it can be addressed in a followup so this is ready!

Noting @Sneha said we should also skip our own onboarding popup, but I might be confusing other people/myself

Will schedule for Monday, May 4th at 4pm eastern

Approval has been secured - moving back to in progress so I don't forget to do it

Taking this from Anne so she can cover for Katie ♻️

Removing myself so people don't think I'm reviewing this - I said in slack that I'd take care of coordinating to make sure this gets deployed on Thursday/next Monday though

🫡🫡🫡

Steph is reviewing (although it seems like @aude is too)

Confirmed everything looks good, thank you!!