Skip to content
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

Remove Developer tab, add Help tab #71

Merged
merged 1 commit into from
Apr 13, 2023
Merged

Remove Developer tab, add Help tab #71

merged 1 commit into from
Apr 13, 2023

Conversation

TimCsaky
Copy link
Contributor

Description

  • Removed Developer tab from navigation. our dev's know how to manually visit the url if we need it
  • add Help tab
  • made navbar always show and hide individual tabs if un/auth'd

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@github-actions
Copy link

Coverage Report (Application)

Totals Coverage
Statements: 75% ( 51 / 68 )
Methods: 62.5% ( 5 / 8 )
Lines: 82.61% ( 38 / 46 )
Branches: 57.14% ( 8 / 14 )

@github-actions
Copy link

Coverage Report (Frontend)

Totals Coverage
Statements: 6.71% ( 56 / 834 )
Methods: 0.52% ( 1 / 194 )
Lines: 9.53% ( 49 / 514 )
Branches: 4.76% ( 6 / 126 )

@kamorel kamorel force-pushed the top-nav branch 2 times, most recently from 2120adb to 8a484b7 Compare April 13, 2023 17:26
@kamorel kamorel merged commit 9f0794d into master Apr 13, 2023
@kamorel kamorel deleted the top-nav branch April 13, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants