-
Notifications
You must be signed in to change notification settings - Fork 212
New Nav Feedback for Version 1 #2663
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
Comments
CC: @Oanh-and-only-Oanh and @sushilshinde |
3,4,5,7,10 are important for our Basic release. |
3 - #2662 |
4:
5 and 7: Ok, can you please tell me, after deploying this code to production, How can I make sure that Payment and forum links would still work?
|
Hi, i was invonved in There is another issue like 10: 13: When I go to Its because These pages doesn't exists in menu. currently when address doesn't exists in menu, menu will expand with first level 1 menu item (if user is logged in), which is business (in this line). default menu for the pages that doesnt exists in menu are handled in this line, the selected menu for this urls can be handled there (currently only About (13.a) : it's because of small bug in here. it will only force level 2 menu to be as same as |
4 - OK |
Thanks @ya3ya6 13: We have community --> My Profile for /members/TonyJ or are you referring to some other page? |
@nithyaasworld No prolem. perhaps because that's your profile, so its marked as |
Got it, Thanks @ya3ya6 I will add this to the original description of this Issue. |
@nithyaasworld No problem, please also check that |
Yes @ya3ya6, it is working just like what we have now in Production. |
|
Thanks @ya3ya6 for reporting!
Added as 13a
Should be fixed in No: 2
Expectation is, when a first-level menu is clicked, it should auto-load the first Second-level item for the selected first level item. This is No: 1 in this list. |
For 10, @Oanh-and-only-Oanh confirmed that we should just keep the Community menu expanded with no selection indicator. |
@sushilshinde, can you help with issues #5 and #7 below? |
@Oanh-and-only-Oanh @luizrrodrigues @nithyaasworld those link should be pointing to, I assume connect dev environment when the build is dev and pointing to production connect if the build is prod. For rest of the links where if there is no corresponding app avail on dev at least URL should form dynamically so if the build is deployed on the beta which will be production build, we should be able to test the whole app on beta. |
@luizrrodrigues Closing this ticket as the issues reported in the description are logged as multiple tickets for your easy tracking. |
These are observed in https://test-community-app.topcoder-dev.com/
Clicking on Compete or Tracks or Explore, not loading the first second-level menu item by default13.a When I go to a challenge details page (challenges/30055052), it loads the page correctly. But the top nav points to Business. .
The text was updated successfully, but these errors were encountered: