Notifications dropdown backend integration

[Oanh-and-only-Oanh]

Change existing mock call to the integration with backend
Challenge link: https://www.topcoder.com/challenges/30117693/?type=develop

[SathyaJayabal]

@sachin-maheshwari @luizrrodrigues , broadcast notifications are not seen for some users.
eg: TonyJ, Mess, tester1234 etc
We can see broadcast notifications for the following user : dan_developer

[SathyaJayabal]

@sachin-maheshwari @luizrrodrigues , for challenge specific notifications, the users do not get the notification when the checkpoint review ends.

[SathyaJayabal]

@sachin-maheshwari @luizrrodrigues , When the challenge spec for the challenge"30055753" is modified, we can observe the following. this is the registrants list:

No notification for dan_developer

Double notification for TonyJ

Correct notification for Mess

[sachin-maheshwari]

@sachin-maheshwari @luizrrodrigues , When the challenge spec for the challenge"30055753" is modified, we can observe the following. this is the registrants list:

No notification for dan_developer

Double notification for TonyJ

Correct notification for Mess

As checked quickly for dan_developer, no notification issue - not getting 'dan_devloper' in corresponding challenge api call i.e. https://api.topcoder-dev.com/v4/challenges/30055753/resources

[luizrrodrigues]

Double notification for TonyJ
@SathyaJayabal this coming from backend:

@sachin-maheshwari

[luizrrodrigues]

@sachin-maheshwari @luizrrodrigues , broadcast notifications are not seen for some users.
eg: TonyJ, Mess, tester1234 etc
We can see broadcast notifications for the following user : dan_developer

I'm able to see to TonyJ user, can you share login info to:
Mess and tester1234

Thanks

[luizrrodrigues]

For challenge specific notifications, the users do not get the notification when the checkpoint review ends.

@sachin-maheshwari I don't see any notification from notifications.autopilot.events yet. Is this the right type to challenge autopilot events right?

[SathyaJayabal]

@sachin-maheshwari @luizrrodrigues , broadcast notifications are not seen for some users.
eg: TonyJ, Mess, tester1234 etc
We can see broadcast notifications for the following user : dan_developer

I'm able to see to TonyJ user, can you share login info to:
Mess and tester1234

Thanks

password for both: appirio123

[SathyaJayabal]

@luizrrodrigues , we had to remove the notifications for submission uploaded event as per. the below discussion.
https://topcoder.slack.com/archives/CG2J9RH70/p1583878405014700?thread_ts=1583858564.012400&cid=CG2J9RH70

On a submission being uploaded, the notification is not displayed, but the pink dot is shown when a submission is uploaded.

[luizrrodrigues]

@SathyaJayabal Thanks for check. This was fixed.
topcoder-platform/navigation-component@af2c926

[SathyaJayabal]

@SathyaJayabal Thanks for check. This was fixed.
topcoder-platform/navigation-component@af2c926

Still see it @luizrrodrigues . See video for reference
sub notif.mov.zip

@sachin-maheshwari @luizrrodrigues , Marking this as fail until all the above issues are fixed. But let me know if you need any specific scenario tested.

[luizrrodrigues]

@SathyaJayabal I have fixed the issue when msg isRead but no isSeen.

Now we need backend update, @sachin-maheshwari we need to remove submission.notification.create type from backend return.

[sachin-maheshwari]

Double notification for TonyJ
@SathyaJayabal this coming from backend:

@sachin-maheshwari

Please check message is same but 'id' is different that mean same message is published twice.

[sachin-maheshwari]

@SathyaJayabal Can please summarize the issues which i need to address.

[SathyaJayabal]

@sachin-maheshwari ,

1. Broadcast notifications not showing up for some users
   eg: TonyJ, Mess, tester1234 etc
   We can see broadcast notifications for the following user : dan_developer
2. Checkpoint review end notification is not displayed for all users.
3. Challenge spec modified notification is displaying inconsistently for different users ( Notifications dropdown backend integration #4055 (comment))
   a. No notification for dan_developer - Here you say dan_developer is not returned from the corresponding challenge api call, But the user is shown as registered and also submitted for that challenge. This could be a data integrity issue with the dev data. What do you suggest we do about this case?
   b. double notification for tonyj

[SathyaJayabal]

3: a is tracked here #4104
3: b. is tracked here #4105

[sachin-maheshwari]

1 is tracked here topcoder-platform/tc-notifications#182

[luizrrodrigues]

@SathyaJayabal create this one too:

Now we need backend update, @sachin-maheshwari we need to remove submission.notification.create type from backend return.

[SathyaJayabal]

@SathyaJayabal create this one too:

Now we need backend update, @sachin-maheshwari we need to remove submission.notification.create type from backend return.

tracked here #4108
cc @luizrrodrigues

[SathyaJayabal]

#2 tracked here #4114

[SathyaJayabal]

#4104
except for this issue everything else works fine.
I am marking this as pass.

[SathyaJayabal]

verified on beta (develop).
Broadcast notifications
Checkpoint review and Checkpoint Screening notifications

Spec modified notification is deferred to release 2.

[SathyaJayabal]

verified on prod