Skip to content

feat:create-navigation-button#227

Open
sammyrolfe wants to merge 2 commits intothoth-tech:9.xfrom
sammyrolfe:create-courseflow-navigation
Open

feat:create-navigation-button#227
sammyrolfe wants to merge 2 commits intothoth-tech:9.xfrom
sammyrolfe:create-courseflow-navigation

Conversation

@sammyrolfe
Copy link

@sammyrolfe sammyrolfe commented Aug 5, 2024

Description

Added option to header dropdown menu for navigation to the coursemap page.

Fixes # (issue)
Screenshot 2024-08-07 at 7 18 09 pm

Screenshot 2024-08-07 at 7 18 11 pm
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested navigation with button and rendering landing page correctly with multiple browsers.

Testing Checklist:

  • [] Tested in latest Chrome
  • [] Tested in latest Safari
  • [] Tested in latest Firefox

Checklist:

  • [] My code follows the style guidelines of this project
  • [] I have performed a self-review of my own code
  • [] I have commented my code in hard-to-understand areas
  • [] I have made corresponding changes to the documentation
  • [] My changes generate no new warnings

@sylvia0929
Copy link

Tested navigation and landing page rendering across multiple browsers.
Checklist items look good. Code style and comments are clear. No new warnings introduced."

@DudeUnleashed
Copy link

Everything tested and works as intended, new button links to the appropriate new page ready for future courseflow.
Code looks good, comments are clear and no new warnings are introduced with this addition.

@aNebula
Copy link

aNebula commented Oct 14, 2024

Good work this trimester @sammyrolfe - time to make your contributions 'lasting'.
Please open an upstream PR with these changes against new/course-flow branch on doubtfire-lms/doubtfire-web.

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.

4 participants