rokwire / events-manager

Events Manager is the administrative interface to events in the Rokwire Platform. Through this web application, users can manage existing events coming from calendars and/or create and manage user events.
Apache License 2.0
1 stars 4 forks source link

[BUG] Events Manager not sending notification #1112

Open minump opened 12 months ago

minump commented 12 months ago

Describe the bug Events manager notification feature is not working. After creating an event, on clicking the blue "Notification" button, filling out the title and message in the pop up and clicking the orange "send message" button, all Illinois app users who have "starred"/"favourited" the event, should have received a push notification. However, the users are not getting a push notification.

To Reproduce Steps to reproduce the behavior:

  1. Set the Illinois App privacy settings to 5.
  2. Create an event from web browser events manager and publish
  3. In the illinois app, go to the event and "star" the event.
  4. From events manager, go to the event and send notification.
  5. See that no push notification is received by the Illinois App user.
  6. Able to reproduce this issue in both DEV and PROD instances.

Found this open issue. Is this related? I do not see an endpoint for /event/eventID/devicetokens in notificationsBB. So not sure how utilizing notificationsBB would be beneficial.

Expected behavior A user with a "starred" event is able to get push notifications.

Debugging The tokens required for the push notification https://github.com/rokwire/events-manager/blob/develop/user_events.py#L767 as of now tries to get from the form / pop up box. However, it should have been from /event/eventID/devicetokens end point here . I see that for notification here https://github.com/rokwire/events-manager/blob/develop/user_events.py#L771 it just returns a 200 status. I also do not see any logs in eventsBB and events manager dev instance in AWS . I thought the logger.info would have some info logged in AWS.

Would connecting the tokens with '/event/<id>/devicetokens', methods=['GET'] request work?

Additional context Add any other context about the problem here.

minump commented 12 months ago

@bingzhang appreciate your input in this issue.

bingzhang commented 12 months ago

For the background information about this from Sandeep:

Notifications used work with the Profile BB. That being retired, Events Manager needs to be updated to use the Notification BB. That functionality is currently not working. You should see some existing GitHub issue in Events Manager related to this.

And there is a previous feature task. https://github.com/rokwire/events-manager/issues/791

minump commented 12 months ago

Ok. sounds good. @vburgett please see the description and comments above.. The feature of sending notifications is not supported by Events Manager now.