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

prefix routes and events to avoid conflicts with the old modules #50

Merged
merged 6 commits into from
Dec 6, 2023

Conversation

ETLaurent
Copy link
Contributor

@ETLaurent ETLaurent commented Dec 5, 2023

Prefixing routes and events fixes conflicts with the old @apostrophecms/piece-type-importer and @apostrophecms/piece-type-exporter modules.

@ETLaurent ETLaurent changed the title prefix routes to avoid conflicts with the old pieces import and expor… prefix routes and events to avoid conflicts with the old modules Dec 5, 2023
@ETLaurent ETLaurent marked this pull request as ready for review December 5, 2023 15:20
@@ -2,5 +2,8 @@
"extends": [ "apostrophe", "plugin:vue/vue3-recommended" ],
"globals": {
"apos": true
},
"rules": {
"max-len": "off"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixing this is just not in the scope of that PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that rule just added after having upgraded the dev deps

@ETLaurent
Copy link
Contributor Author

@ETLaurent ETLaurent requested a review from boutell December 5, 2023 16:03
@ETLaurent
Copy link
Contributor Author

"eslint-plugin-node": "^11.1.0",
"eslint-plugin-promise": "^6.1.1",
"eslint-plugin-standard": "^5.0.0",
Copy link
Contributor Author

@ETLaurent ETLaurent Dec 6, 2023

Choose a reason for hiding this comment

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

deprecated, keeping it made npm i fail

@ETLaurent ETLaurent merged commit 31d0732 into main Dec 6, 2023
9 checks passed
@ETLaurent ETLaurent deleted the pro-5017-rename-labels-csv branch December 6, 2023 10: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.

2 participants