Skip to content

Update "Join Community" link in footer #5101

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.

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

Open
SathyaJayabal opened this issue Oct 13, 2020 · 15 comments
Open

Update "Join Community" link in footer #5101

SathyaJayabal opened this issue Oct 13, 2020 · 15 comments
Assignees
Labels
Auth0 fix-after-release P4 Low (resolve within 2 weeks)

Comments

@SathyaJayabal
Copy link
Collaborator

Join community link in the footer must point to new auth0 registration page
Screenshot 2020-10-13 at 10 07 23 PM
cc @sachin-maheshwari

@sachin-maheshwari
Copy link

@luizrrodrigues please fix on priority. We are targeting prod release by next week (20 Oct 2020).

@sandhiyakavi
Copy link
Collaborator

@luizrrodrigues @sachin-maheshwari If user is logged in and click Join Community, 404 HTTP Error is displayed.(redirected to url https://www.topcoder-dev.com/challenges&utm_source=community)

image
Joincommunity_404_error.zip

@sandhiyakavi sandhiyakavi added QA Fail QA verification on Dev has failed. Assignee to redo the fix. and removed Ready for QA labels Oct 14, 2020
@luizrrodrigues
Copy link
Collaborator

@sandhiyakavi I can't reproduce this issue, can you try clear cookies?

2020-10-14 11-16-42.zip

@luizrrodrigues
Copy link
Collaborator

cc @SathyaJayabal

@luizrrodrigues
Copy link
Collaborator

luizrrodrigues commented Oct 15, 2020

@SathyaJayabal @sandhiyakavi I'm able to reproduce now.

@sachin-maheshwari This issue coming from: https://github.com/topcoder-platform/tc-auth-lib/blob/b45d1fa474f3df217d1808c798fd771fc3405868/web-assets/js/setupAuth0WithRedirect.js#L46

let returnAppUrl = qs['retUrl'];
...
...
returnAppUrl += '&utm_source=' + utmSource;
...
...
if (isAuthenticated) {
    window.location = returnAppUrl;
} 

Missing ? between URL and params, I think can fix this with:

const returnAppUrl = new URL(qa['retUrl']);
...
...
returnAppUrl.searchParams.append('utm_source', utmSource);
...
...
if (isAuthenticated) {
    window.location = returnAppUrl.href;
} 

@sachin-maheshwari
Copy link

@LieutenantRoger good catch 👍

@sachin-maheshwari
Copy link

sachin-maheshwari commented Oct 15, 2020

@sushilshinde just curious, what is use of passing parameters (utm_source=community&utm_campaign=tc-footer&utm_medium=promotion) -https://accounts-auth0.topcoder-dev.com/?utm_source=community&utm_campaign=tc-footer&utm_medium=promotion&retUrl=https%3A%2F%2Fwww.topcoder-dev.com%2Fchallenges
Still we are using for any tracking ? can we remove utm_source ?

@sachin-maheshwari
Copy link

fixed. I restricted domain change in "auth lib" for our enterprise customer only i.e. wipro, zurich and cs.

@luizrrodrigues luizrrodrigues removed the QA Fail QA verification on Dev has failed. Assignee to redo the fix. label Oct 18, 2020
@SathyaJayabal SathyaJayabal added Need clarification Need clarification to proceed fixing the issue further tcx_Feedback and removed Ready for QA labels Oct 19, 2020
@SathyaJayabal
Copy link
Collaborator Author

@luizrrodrigues

  1. Not logged in user: When clicking on Join community, the signup tab must be selected by default. (In production, Join community takes the user to the Join topcoder registration page)

Screenshot 2020-10-19 at 10 09 44 AM

  1. Logged in user : When a logged in user clicks on "Join community" , the page is redirected to the same page, without opening the signup page (In production, even when the user is logged in Join Topcoder registration page is displayed).
    5101.mov.zip

@Oanh-and-only-Oanh , what should be the expected behaviour here ?

@Oanh-and-only-Oanh
Copy link

@SathyaJayabal,

  1. correct on this behavior
  2. If the user is logged in, the "join community" link in the footer should not be visible.

@SathyaJayabal
Copy link
Collaborator Author

@luizrrodrigues , can you make the change for the below pls?

@SathyaJayabal,

  1. correct on this behavior
  2. If the user is logged in, the "join community" link in the footer should not be visible.

@SathyaJayabal SathyaJayabal removed the Need clarification Need clarification to proceed fixing the issue further label Oct 20, 2020
@luizrrodrigues
Copy link
Collaborator

  1. I think currently auth lib not support that, @sachin-maheshwari can we add this feature? Maybe if provide regSource show SignUp by default?

  2. Fixed.

@sachin-maheshwari
Copy link

@mtwomey Need your input here. As we discussed, i think, current scenario is fine.

@SathyaJayabal
Copy link
Collaborator Author

verified 1.
Screenshot 2020-10-22 at 11 00 53 AM

@mtwomey
Copy link
Contributor

mtwomey commented Oct 22, 2020

@sachin-maheshwari We might be able to fix, but let's keep this low priority, we can review it after the release.

The idea is that if the use is not logged in and they click join community, the auth window should open, but it starts on the register tab, that's the thought - correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auth0 fix-after-release P4 Low (resolve within 2 weeks)
Projects
None yet
Development

No branches or pull requests

7 participants