Skip to content

Notifications: Links to challenge from notifications must be in new format #4433

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
SathyaJayabal opened this issue May 28, 2020 · 4 comments
Closed

Notifications: Links to challenge from notifications must be in new format #4433

SathyaJayabal opened this issue May 28, 2020 · 4 comments
Labels
Beta Env Environment CF Reviewed - Requirement changed Notifications P1 Urgent (resolve within 24 hours) QA Pass tcx_FixAccepted Test Env Environment v5-intgration-sub-code

Comments

@SathyaJayabal
Copy link
Collaborator

User cred: jcori/appirio123

  1. Go to Notifications page https://test-community-app.topcoder-dev.com/notifications
    or

  2. Open notifications popup from header.

  3. Click on challenge related notification

expected url format:https://test-community-app.topcoder-dev.com/challenges/2b8e84ea-656c-4ee5-bf1d-7030b50d2e9a
actual url format:https://test-community-app.topcoder-dev.com/challenges/30048648

notif.mov.zip
@SathyaJayabal SathyaJayabal added P1 Urgent (resolve within 24 hours) Test Env Environment Notifications v5-intgration-sub-code labels May 28, 2020
@sushilshinde
Copy link
Collaborator

This will be done from the backend as well as #4435

@sushilshinde sushilshinde added the QA Blocked - Dependency QA Verification blocked for another Fix label May 28, 2020
@SathyaJayabal SathyaJayabal added dependent-on-another-issue-fix and removed QA Blocked - Dependency QA Verification blocked for another Fix labels Jun 1, 2020
@sachin-maheshwari
Copy link

@SathyaJayabal @sushilshinde FYI,
As discussed with @mtwomey, once the issue #4435 get fixed, this issue priority automatically will down from P1.

@SathyaJayabal
Copy link
Collaborator Author

SathyaJayabal commented Jun 6, 2020 •
edited
Loading

@sushilshinde , not able to verify as the example challenge has not been migrated
notif.mov.zip

Screenshot 2020-06-06 at 5 05 18 PM

topcoder-platform/challenge-api#149

@SathyaJayabal
Copy link
Collaborator Author

verified on beta
notif.mov.zip

@sushilshinde , we can close this. tcx_accepted label needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Beta Env Environment CF Reviewed - Requirement changed Notifications P1 Urgent (resolve within 24 hours) QA Pass tcx_FixAccepted Test Env Environment v5-intgration-sub-code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@sushilshinde @sachin-maheshwari @lakshmiathreya @SathyaJayabal