Skip to content

$30 | New Nav: Regarding clicking on Topcoder Logo #104

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
nithyaasworld opened this issue Jul 1, 2019 · 11 comments
Closed

$30 | New Nav: Regarding clicking on Topcoder Logo #104

nithyaasworld opened this issue Jul 1, 2019 · 11 comments
Assignees
@gets0ul
Labels
nav_Assigned nav_FixAccepted nav_Paid P3 Normal (resolved within 7 days) Prod Env Environment QA Pass
Milestone
Community App 0.2...

Comments

@nithyaasworld
Copy link
Collaborator

For V2, the logo should direct them to "my Dashboard" if they're logged in. and www.topcoder.com if they are logged out
@kkartunov
Copy link
Contributor

topcoder-platform/community-app#2708 is related. Let's make the link configurable by component property! Thanks, @luizrrodrigues

@luizrrodrigues luizrrodrigues removed their assignment Jan 9, 2020
@luizrrodrigues luizrrodrigues changed the title New Nav: Regarding clicking on Topcoder Logo $30 | New Nav: Regarding clicking on Topcoder Logo Jan 9, 2020
@ghost
Copy link

ghost commented Jan 9, 2020

I would like to work on this issue.

@luizrrodrigues luizrrodrigues assigned ghost Jan 9, 2020
@luizrrodrigues
Copy link
Collaborator

@fctorial Any status in this one?

@ghost
Copy link

ghost commented Jan 10, 2020 via email

@luizrrodrigues luizrrodrigues unassigned ghost Jan 10, 2020
@nahidshahin nahidshahin self-assigned this Jan 10, 2020
@nahidshahin
Copy link
Contributor

@luizrrodrigues MR #92

@luizrrodrigues luizrrodrigues transferred this issue from topcoder-platform/community-app Jan 14, 2020
@luizrrodrigues luizrrodrigues added nav_Feedback nav_Assigned P3 Normal (resolved within 7 days) labels Jan 14, 2020
@luizrrodrigues
Copy link
Collaborator

@nahidshahin Please check feedback here: #92 (review)

@luizrrodrigues
Copy link
Collaborator

@nahidshahin any status in this one?

@luizrrodrigues
Copy link
Collaborator

@nahidshahin We opening this one to pickup, please reassign if you able to complete the fix.

Thanks.

@gets0ul gets0ul self-assigned this Jan 16, 2020
This was referenced Jan 16, 2020
Merged
Merged
luizrrodrigues added a commit that referenced this issue Jan 22, 2020
@luizrrodrigues
Merge pull request #125 from gets0ul/issue_104
e9dc59a 
Fix for Issue #104 - Logo Link
@luizrrodrigues luizrrodrigues mentioned this issue Jan 22, 2020
Closed
@nithyaasworld
Copy link
Collaborator Author

Verified in Test env.

@macs054
Copy link
Collaborator

macs054 commented Feb 27, 2020

Verified in Beta
issue_104_beta.zip

@macs054 macs054 added Beta Env Environment and removed Test Env Environment labels Feb 27, 2020
@SathyaJayabal SathyaJayabal added this to the Community App 0.21.01 milestone Feb 27, 2020
@SathyaJayabal
Copy link
Collaborator

verified in prod
logo.mov.zip

@luizrrodrigues luizrrodrigues added Prod Env Environment and removed Beta Env Environment labels Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gets0ul gets0ul

Labels
nav_Assigned nav_FixAccepted nav_Paid P3 Normal (resolved within 7 days) Prod Env Environment QA Pass
Projects
None yet
Milestone
Community App 0.21.01
Development

No branches or pull requests
7 participants
@macs054 @nahidshahin @gets0ul @nithyaasworld @kkartunov @luizrrodrigues @SathyaJayabal