juho-jaakkola / elgg-chat

Chat-like messaging plugin for Elgg
10 stars 6 forks source link

Site notifications are broken #7

Open juho-jaakkola opened 9 years ago

juho-jaakkola commented 9 years ago

@Pasley70 said:

The recipients site notification - on opening a new chat - has no title and link to the chat, but just pointing "A new Chat" with a "broken" icon. It would be useful, if the recipient get the same information on the site as in the E-Mail. Maybe a conflict with new site notification in 1.9?

juho-jaakkola commented 9 years ago

I'm not sure if site notifications are needed at all. After all the chat icon in the topbar notifies user about unread chat messages. I think I'd prefer disabling site notifications completely.

What do you think?

Pasley70 commented 9 years ago

Yes, you are right. Site-Notifications are redundant. I agree with your proposal.