Notification: Link in broadcast messages

[SathyaJayabal]

Expected: Clicking the broadcast message must take the. user to the respective page

Actual: The link is displayed as part of the broadcast message

cc @sachin-maheshwari

[luizrrodrigues]

@Oanh-and-only-Oanh Please confirm this.

We going to use the link directly in the Notification body with the highlighted test (in e.g. above the Topcoder will be blue like "View all Notifications". Or we'll make the whole notification item as a link, as we have in Challenge Notifications type?

[Oanh-and-only-Oanh]

@luizrrodrigues, the notification item should take them to the respective page.

[luizrrodrigues]

@Oanh-and-only-Oanh Yep, this requirement is okay.

Question is about if not will show link in notification body level, like the example above (screenshot) only Topcoder will be a link.

[luizrrodrigues]

If follow example above, notifications with a link will show like this:

[Oanh-and-only-Oanh]

@luizrrodrigues, see below

[luizrrodrigues]

Okay, I just update Sachin:3 public group testing Topcoder to confirm :)

I think not a better way, think about we add a new field to API response with the URL and make whole notification item as a link, like challenge items.

But this change will need to do in the backend, so we can do this later.

I'll make the updates like the request above.

[luizrrodrigues]

@SathyaJayabal I'm done the implementation and build is running (ready to QA in 20min).

But need to fix links in notifications items.

Need to change herf to href

[luizrrodrigues]

The build is done:

[SathyaJayabal]

verified on test env

[SathyaJayabal]

Verified by @sushilshinde beta(feature branch)

[SathyaJayabal]

can be verified on production only when a broadcast message with a link is published.