Grasia / teem

Teem is a web tool and mobile app for collaboration in communities, which facilitates onboarding of newcomers
http://teem.works/
GNU Affero General Public License v3.0
49 stars 25 forks source link

Mobile notifications on a teem are delivered to community participants #156

Closed atd closed 8 years ago

atd commented 8 years ago

Notifications from a chat message are delivered to community participants. They should be delivered to participants of that teem only

atfornes commented 8 years ago

Since we changed SwellRT subscritions to be managed as blacklist (https://github.com/P2Pvalue/swellrt/commit/951cfe7775ad4079edd8da1a855afa3a81175324), an user has to unsubscribe from every wave she visits not to receive notifications.

We could try to add a condition in SwellRT event rules, but current implementation does not allow to check for the membership of an item in a list, making it imposible handle from here. See https://github.com/P2Pvalue/swellrt/blob/master/src/org/swellrt/server/box/events/EventRule.java#L252.

I am waiting for @pablojan, @atd and @llopv opinions on this before continuing working in this issue.

atd commented 8 years ago

What was the rationale behind subscribing visitors to waves? I have been lately wondering why we had two lists for (what I though) was the same issue. In other words, I expected wave participants to be the same as project participants

atd commented 8 years ago

Fixed in SwellRT