Page MenuHomePhabricator

Shorten and break up the LoginNotify notification
Closed, ResolvedPublic

Description

We are fixing T163518, to stop truncating bundles. But especially given that change, please shorten the LoginNotify message, below:

There have been 6 failed attempts to login to your account since the last time you logged in. If this was you, then you can disregard this message. if it wasn't, please make sure your account has a strong password.

I can see two easy ways to do this:

  1. separate the message into message header vs. message body, which is proper style anyway. That way in a bundle, you will see only the header. Which, in this case, would be the first sentence.
  1. Drop the second sentence, which is not really necessary so not appropriate to this compressed environment.

Header: There have been 6 failed attempts to log in to your account since the last time you logged in.
Body: If this was't you, then please make sure your account has a strong password.

(Also, btw, "log in" as a verb should be two words.)

Event Timeline

We've updated the notification to be:

There have been 6 failed attempts to log in to your account from a new device since the last time you logged in. If it wasn't you, please make sure your account has a strong password.

Does that resolve this ticket?

Great! It's definitely better. Do I take it you don't want to split the message up between header and body?

Great! It's definitely better. Do I take it you don't want to split the message up between header and body?

Hmm, I'm not clear what you mean by header message and body message. Since it's a cross-wiki bundle, there doesn't seem to be a reliable way to show them the complete message, no matter what wiki they are on (besides shortening it and showing the whole thing).
Marking the bundle as read splits it into shorter, generic messages like "There have been multiple failed attempts to log in to your account from a new device. Please make sure your account has a strong password."" to avoid long, duplicate messages which would feel more spammy.

Hmm, I'm not clear what you mean by header message and body message.

There are two possible parts to any notification. The header, in bold, and the body, in a light gray text—usually used for excerpts. If you want the user to see the whole thing in a bundle, then you're right, the body wouldn't be shown. So, yeah, that seems like a good change.

kaldari claimed this task.
kaldari moved this task from New & TBD Tickets to Archive on the Community-Tech board.