$20 | New nav - hide white secondary menus when empty

[kkartunov]

If some of:

• https://github.com/topcoder-platform/community-app/blob/25b95419ac4f14f36c186d6a7e04312e565a5ce4/config/default.js#L323
• https://github.com/topcoder-platform/community-app/blob/25b95419ac4f14f36c186d6a7e04312e565a5ce4/config/default.js#L308

is empty do not show the corresponding menu element/area.

[rashmi73]

@luizrrodrigues any scenario where it becomes empty?

[luizrrodrigues]

@rashmi73 You need to edit config file:

config/default.js
config/production.js

[luizrrodrigues]

@rashmi73 Any status on this one?

[luizrrodrigues]

@gets0ul Not working to subMenu:

[gets0ul]

@luizrrodrigues Can you bump the prize because the original requirement was for secondaryMenu only, not including subMenu?

[gets0ul]

@luizrrodrigues PR is updated.

[luizrrodrigues]

@gets0ul I'll check about prize.

[luizrrodrigues]

@gets0ul Prize updated

[luizrrodrigues]

@gets0ul Need hide secondary menu and arrow from mobile too:

[gets0ul]

@luizrrodrigues PR updated for mobile menu #105

[luizrrodrigues]

@gets0ul still see some issues with your approach, like still seeing arrow and in menu when submenu not exist. But based in your approach I have fixed other issues, please take a look: https://github.com/topcoder-platform/navigation-component/pull/146/files

[SathyaJayabal]

@luizrrodrigues @kkartunov How can we reproduce / test this issue ?

[luizrrodrigues]

@SathyaJayabal I'll add video verification here.

[gets0ul]

@luizrrodrigues Why is it not getting into 'Paid' status?

@SathyaJayabal To test, you need to set the submenu to be empty #142 (comment)

[luizrrodrigues]

@gets0ul Sorry delay here, I'll provide a video to QA team pass this one.

[luizrrodrigues]

@SathyaJayabal Please check videos below:
https://drive.google.com/open?id=1crnhjLtmSXWLoy_MgD55x7ck4J96NKX3
https://drive.google.com/open?id=1EcLpgarPo9hX8FoDk32eKMPKUoUeljmP

[SathyaJayabal]

@luizrrodrigues , thank you for the videos.

[SathyaJayabal]

@luizrrodrigues , please confirm that this cannot be verified on beta.

[luizrrodrigues]

@SathyaJayabal Yes, the verification need be via local env, like shared in the video.

[SathyaJayabal]

Cannot be verified in production.