Page MenuHomePhabricator

2019 Week 6 Bonus Content
Closed, ResolvedPublic

Description

PHI1050 identified some out-of-date documentation.

D20080 has an instance service call which should receive jitter.

PHI1047 notes an API inconsistency between owners.search and owners.edit.

User renames show up in feed but don't have a feed-oriented rendering ("this user" instead of "<old username">): https://secure.phabricator.com/feed/6654516638475651570/

The "Members" UI under instances still has a couple of issues:

  • Invites which were cancelled but ultimately registered (through automatic domain registration? Old invites? Is this just a bug?) show as "cancelled", but should show in a user bucket.
  • Users can be instance managers and in a lot of other states (Unregistered, Disabled). Our icon signaling isn't terribly clear in these cases.
  • Unregistered users should show up as "invited".

D20093 fixes a mobile application menu in Mail. It would be very nice to fix this in a generic way instead. (This may also affect Instances.)

See PHI1055, which would like more sophisticated Owners Audit behavior.

PHI1034 is a cluster invoicing thing which should be sorted out.

PHI1052 encountered a card issue with Stripe. This might be a problem with the card, but the workflow should be more clear.

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
rP Phabricator
D20127
D20131
D20132
D20130
D20126
D20124
D20129
D20102
D20100
D20096
D20095
D20094
D20097
D20093
D20091
D20087

Event Timeline

epriestley triaged this task as Normal priority.Feb 1 2019, 10:38 PM
epriestley created this task.
epriestley added a revision: Restricted Differential Revision.Feb 5 2019, 3:15 AM
epriestley added a revision: Restricted Differential Revision.Feb 5 2019, 3:17 AM
epriestley added a revision: Restricted Differential Revision.Feb 5 2019, 6:29 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 5 2019, 10:23 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 6 2019, 12:03 AM
epriestley added a commit: Restricted Diffusion Commit.Feb 7 2019, 3:25 PM
epriestley added a commit: Restricted Diffusion Commit.

D20093 fixes a mobile application menu in Mail. It would be very nice to fix this in a generic way instead. (This may also affect Instances.)

This is at least slightly involved: although we can make the menu generate automatically for pages which have a legitimate left-hand-side navigation menu, we also currently set things up so that detail pages (like /V123) have the full menu.

I'm sort of not sure we really need or want to do this. We don't have these options anywhere on Desktop, and that's fine: you just click the "Slowvote" breadcrumb to get back to the list. I think you rarely want to jump directly from /V123 to "Advanced Search" or some other non-default filter.

One mild argument is that we sometimes abbreviate breadcrumbs on mobile, but very few pages have some complex A > B > C > D hierarchy.

I'm leaning toward making this automatic in the easy case: when we have a real left-side navigation menu, always make it the default "hamburger" mobile menu on the same page. And then maybe moving toward getting rid of it in (most?) other cases? The new rule would be "if there's a left-hand navigation menu on Desktop, you get the same hamburger menu on mobile; if there's no left-hand navigation menu on Desktop, you get nothing".

For now, maybe I'll just keep an eye on how that rule would impact things.

For now, maybe I'll just keep an eye on how that rule would impact things.

Immortalized in T13247 since this isn't immediately resolvable.

epriestley claimed this task.
epriestley updated the task description. (Show Details)