-
-
Notifications
You must be signed in to change notification settings - Fork 54
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
[FEAT] Ignore all content types but the created by content type builder, i.e., api content types #167
Comments
Hi @SalahAdDin Thanks for reporting yet another great feature request! About the sitemap plugin; we are currently in the process of migrating the plugin to the Webtools suite. The issue you created about the "Ignore all content types..." is a feature that's already integrated in Webtools. That being said I would love it if you could give Webtools and the Sitemap addon a try. I think you'll like it a lot. |
There are some issues with this, the first, it is not clear if we need to install This happens when we don't setup the configuration, when we do it, it fails even to load the admin: ~/Projects/Portfolio/backend PBE-001-strapi-4 !4 yarn develop ✔ 20s system
✔ Cleaning dist dir (10ms)
⠋ Building build context
[INFO] Including the following ENV variables as part of the JS bundle:
- ADMIN_PATH
- STRAPI_ADMIN_BACKEND_URL
- STRAPI_TELEMETRY_DISABLED
✔ Building build context (107ms)
✔ Creating admin (9161ms)
⠙ Loading Strapi[ERROR] There seems to be an unexpected error, try again with --debug for more information
┌───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│ │
│ Error: Error loading the plugin sitemap because sitemap is not installed. Please either install the plugin or remove it's configuration. │
│ at Object.loadPlugins (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/strapi/dist/core/loaders/plugins/index.js:83:13) │
│ at async Strapi.loadPlugins (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/strapi/dist/Strapi.js:311:5) │
│ at async Promise.all (index 3) │
│ at async Strapi.register (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/strapi/dist/Strapi.js:341:5) │
│ at async Strapi.load (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/strapi/dist/Strapi.js:425:5) │
│ at async Object.develop (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/admin/dist/_chunks/index-sNH2VWbC.js:837:28) │
│ at async develop (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/admin/dist/_chunks/develop-YNTGWKMC.js:61:5) │
│ at async Command.parseAsync (/home/luisalaguna/Projects/Portfolio/backend/node_modules/commander/lib/command.js:923:5) │
│ at async Module.runStrapiCommand (/home/luisalaguna/Projects/Portfolio/backend/node_modules/@strapi/strapi/dist/commands/index.js:130:3) │
│ │
└───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ Installing web tools works fine, but the sitemap cannot be enabled since the config is still broken. So, it is not usable yet. I think it is required to update the documentation! |
Could you report an issue on the webtools repo about this? |
Opened: pluginpal/strapi-webtools#150 |
I'm going to close this issue. This standalone Sitemap plugin will be deprecated when it's successor Webtools will be out of Beta. And seeing as this particular issue is irrelevant when using Webtools, there seems no reason to leave this issue open. |
Feature request
Summary
Checking the sitemap configuration for the setup of the URLs we find the following content-type hell:
It shows a lot of content types that we don't want to make part of any website.
We manually excluded them, but they are too much for the beginning, and no, it does not only include third-party plugin types:
Why is it needed?
As you can see, it is hard for the developer tool and check which content types we need to exclude from the sitemap one by one.
Suggested solution(s)
It would be better either to add a new property
includeType
or make the plugin check only the API content types, so things get easier, even more, it should only include those types and exclude the types we want from API content types since all plugins and Strapi content types are not public in the most of the cases.As you can see from the comment
yarn strapi content-types:list
, the only content types we want to see are the following:Related issue(s)/PR(s)
Let us know if this is related to any issue/pull request.
The text was updated successfully, but these errors were encountered: