search

dhsanket / emote

emote afresh
2 stars 1 forks source link

In-app Notifications for users - informing about updates to Titles they previously emoted on, or favourited #43

Open dhsanket opened 10 years ago

ostadhashem commented 10 years ago

As I found, we have Title domain that contains followed user, so we can use it for any changes in title, and if a title changed, we save a notification for all of the users who follow this title. and in each fetch for any user, it gets all of the notifications for this user, and flag it as read. I think it's better that we remove notified item from this domain and put it in another too. The other problem, is that we don't save favorite item in this manner, and this query has a performance problem, I think that we can save this duplication in Title and FavoriteUser and in add and remove , do it in both of them. I will be glad for any advise.

nitinkarmuse commented 10 years ago

Can you elaborate what are two duplicates here?

ostadhashem commented 10 years ago

I think that we can start it without duplication, but if we found that it has some performance issues to fetch it from nested array, we can add faroriteForUsers property to Title domain object too.