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

Rename vues to multiword #118

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

pstaabp
Copy link
Member

@pstaabp pstaabp commented Aug 28, 2022

In a more recent version of vue-eslint, there is a requirement that vue pages and components to be multiword. This is designed not conflict with future html tags. See https://eslint.vuejs.org/rules/multi-word-component-names.html

This changes no functionality, just renaming files and updating affected components.

This relies on #106, which has some UI changes. We should push this through quickly so any updates on components won't conflict in two files.

WIP: work on course_settings

WIP: continued work on the course settings

WIP: continued work on course_settings.

TEST: mojolicious route testing for settings.

WIP: continued work on course settings
perltidy
WIP: allow default settings to be defined in an override file.
WIP: fix up tests and general cleanup.


WIP: fix up tests and general cleanup.


FIX: errors after rebase
WIP: switch pages/components to multi-word
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.

1 participant