Skip to content

Exclude sidekick assessment#16

Open
cuzuco2 wants to merge 5 commits intomainfrom
exclude-sidekick
Open

Exclude sidekick assessment#16
cuzuco2 wants to merge 5 commits intomainfrom
exclude-sidekick

Conversation

@cuzuco2
Copy link
Collaborator

@cuzuco2 cuzuco2 commented Apr 8, 2024

Added excludeSidekick assessment


async function checkNoIndexMetaTag(url) {
try {
const response = await fetch(url);
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we only need to check the root url?

Copy link
Collaborator Author

@cuzuco2 cuzuco2 Apr 8, 2024

Choose a reason for hiding this comment

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

@dzehnder This is the library page, that being said that was the only page I found, if you know of any other let me know

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

As we have seen the other ones to check are the ones in library.json, I'll check that

@dzehnder
Copy link
Collaborator

dzehnder commented Apr 8, 2024

Some additional comments from meeting with Alvaro:
How can we check all possible URLs that relate to sidekick?
Solution for now: we scan the sidekick plugins listed in the config.json
To get the URLs for the Block library (if it is listed as a plugin) there is a library.json file

Some additional questions that came up:
Should any of the sidekick pages even exist on a prod page (with noindex) or should it not be published in the first place?
Is it enough to have noindex on root of the sidekick plugins or does every subfolder/plugin need the noindex propery?

cuzuco2 added 2 commits April 8, 2024 17:41
…uplicated -export function and added to all-assessments -use of HttpClient instead of fetch
@cuzuco2
Copy link
Collaborator Author

cuzuco2 commented Apr 8, 2024

Maybe for the library page we could use also nofollow, to be sure that the block pages are not indexed, but that would mean that this is the only way for those pages to be discovered (which we are not sure)

But then again, maybe in the first place this pages shouldn't be out there published at all

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