Include account name in notification

[ThHareau]

Hello,

Would it be possible to include the account name in the notification?

The index of the account is not really helpful.

I think that the main email attached to the account would be more useful (something like: email@domain.tld: 24 new emails).

[vladimiry]

Initially, the login name was displayed in the notification message but then got turned to the index due to the privacy matters. We didn't have the aliasing feature at that time but we got aliasing supported since v4.0.0 and so I think it's ok to render the alias in the notification message. If no alias value defined then it would still be the index. Any thoughts on the described scenario?

[ThHareau]

For me at least, alias is perfect.

[vladimiry]

Alias displaying in desktop notification message is going to be enabled in the next release, 04fc659.

[ThHareau]

That's awesome @vladimiry , I didn't expected it to be implemented that fast!
Thanks a lot

[vladimiry]

That was very easy change since we already had all the data in the store.

[vladimiry]

You can find in this message a link to the most actual work-in-progress build with 04fc659 change included.