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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Nc prefix to all components #3058

Merged
merged 1 commit into from
Aug 18, 2022
Merged

Conversation

raimund-schluessler
Copy link
Contributor

@raimund-schluessler raimund-schluessler commented Aug 17, 2022

This adds the Nc prefix to all component names to be in line with eslint and consistent for all components.

This was not a fun thing to do, and won't be when migrating apps 馃檲

Closes #2643, supersedes #3050

@raimund-schluessler raimund-schluessler added 2. developing Work in progress component Component discussion and/or suggestion technical debt breaking PR that requires a new major version labels Aug 17, 2022
@raimund-schluessler raimund-schluessler force-pushed the fix/2643/component-names branch 5 times, most recently from 03dc636 to 5e46fce Compare August 18, 2022 07:38
@raimund-schluessler raimund-schluessler marked this pull request as ready for review August 18, 2022 07:46
@raimund-schluessler raimund-schluessler added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 18, 2022
@raimund-schluessler raimund-schluessler added this to the 6.0.0 milestone Aug 18, 2022
Copy link
Contributor

@CarlSchwan CarlSchwan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot

Copy link
Contributor

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an elephant in my room

@skjnldsv
Copy link
Contributor

Conflicts!

@nickvergessen
Copy link
Contributor

We should stop merging stuff until this is in.

Conflict caused by a one liner
#3059

Signed-off-by: Raimund Schl眉脽ler <raimund.schluessler@mailbox.org>
@raimund-schluessler
Copy link
Contributor Author

Rebased.

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 18, 2022
@nickvergessen nickvergessen merged commit 063d87e into master Aug 18, 2022
@nickvergessen nickvergessen deleted the fix/2643/component-names branch August 18, 2022 12:30
Oliv4945 added a commit to Oliv4945/ldapcontacts that referenced this pull request May 15, 2023
So renamed all Nextcloud component with NC prefix, see [here](nextcloud-libraries/nextcloud-vue#3058)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish breaking PR that requires a new major version component Component discussion and/or suggestion technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename Vue components to multi word components
5 participants