Closed kodyVS closed 10 months ago
Did you want events and resources in both menu's?
I'm out of the loop for the discussions referenced, but this looks good to me 👍
@kodyVS sorry I gave you merge conflicts, once we figure out what actually goes into the menu then this looks good from me too
No problem.
The navigation in the home sub menu goes to the sub-headers on the main page and the main links go to the full pages.
We could rename the home sub menu sections to get rid of any confusion instead of "Meetups, events and resources" we have "Community" and add "Newsletter" to the list.
Ooooohhhh woops, my bad, you are right. Yeah this looks good to me!
Oh one more question, is there a reason you deleted all the unit tests? I think they are still valid ones?
I deleted the unit tests because I changed the way the menu is displayed to utilize the built-in features of vue and tailwind and in doing so, changed the methods that were in the test. However, I can add 2 more unit tests for the methods I replaced.
Thank you so much for adding those! I appreciate it!
Sorry my changes are now breaking your imports :)
@all-contributors add @kodyVS for code and ideas
@MandyMeindersma
I've put up a pull request to add @kodyVS! :tada:
Thank you so much Kody!
What issue is this referencing?
No issue reference but adding changes from the discussion
Do these code changes work locally and have you tested that they fix the issue yourself?
Does the following command run without warnings or errors?
npm run pr-checks
Have you taken a look at our contributing guidelines?
My node version matches the one suggested when running
nvm use
?