telefonicaid / fiware-orion

Context Broker and CEF building block for context data management, providing NGSI interfaces.
https://fiware-orion.rtfd.io/
GNU Affero General Public License v3.0
212 stars 264 forks source link

Googling FIWARE topics brings up outdated documentation #4218

Open Maghnie opened 1 year ago

Maghnie commented 1 year ago

Bug description Not sure if this is the best place for this "bug report", because this is more of a "bug" in the user experience with googling FIWARE documentation. For example, when I googled a FIWARE concept, the top result led to a seemingly outdated documentation page. (screenshot below)

How to reproduce it Since Google search results are personalized, not sure how this can be reproduced in a stable way.

Expected behavior When googling for a FIWARE concept, only the updated docs should show up in the results, or the outdated docs could redirect to the newest docs. So in the example above, instead of: https://fiware-orion.readthedocs.io/en/1.3.0/user/federation/index.html , this page should show up in Google: https://fiware-orion.readthedocs.io/en/master/user/federation.html

Additional information

image

fgalan commented 1 year ago

As far as I know the Google algorithm show first the pages that have many other pages linking to them. Thus, for that particular example, I guess that the document https://fiware-orion.readthedocs.io/en/1.3.0/user/federation/index.html is more linked that the https://fiware-orion.readthedocs.io/en/master/user/federation.html

To change this behaviour we should be able to:

As you can see, this is completely out of the scope of our rearch.

fgalan commented 1 year ago

I would like to take this opportunity to tell you that documentation about the Orion NGSIv2 API has been recently restructured and improved, available at this link:

https://github.com/telefonicaid/fiware-orion/blob/master/doc/manuals/orion-api.md

Please, update your bookmarks with it.

Maghnie commented 1 year ago

I'm not sure either how to block Google from showing outdated pages, but there might be a way with the "noindex" meta tag: https://developers.google.com/search/docs/crawling-indexing/block-indexing#:~:text=You%20can%20prevent%20a%20page,other%20sites%20link%20to%20it.

Another solution, possibly simpler, would be to display a sort of warning at the top of old documentation pages, while linking to the latest docs, as scipy does here: https://docs.scipy.org/doc/scipy-1.3.1/reference/

fgalan commented 1 year ago

Another solution, possibly simpler, would be to display a sort of warning at the top of old documentation pages, while linking to the latest docs, as scipy does here: https://docs.scipy.org/doc/scipy-1.3.1/reference/

I understand that this is already happening. For instance if you go to https://fiware-orion.readthedocs.io/en/3.1.0/ you'll see:

imagen

Maghnie commented 1 year ago

Yes, that looks great. Something like that would be perfect to have on all outdated doc pages.

It seems that docs with a version less than 2 don't show this warning, e.g.: https://fiware-orion.readthedocs.io/en/1.15.0/user/multitenancy/index.html

fgalan commented 1 year ago

It seems that docs with a version less than 2 don't show this warning, e.g.: https://fiware-orion.readthedocs.io/en/1.15.0/user/multitenancy/index.html

Not sure why it works that way... we have no control on how readthedocs renders their pages. Maybe you could open an issue at readthedocs about that, please? In that case, please include the link to that issue as comment here.

Thanks in advance!

fgalan commented 1 year ago

After the discussion on https://github.com/readthedocs/readthedocs.org/issues/9633 if you tell me which versions are not correct I can relaunch the build for all them. Thanks!

Maghnie commented 1 year ago

It seems that all versions before 2.0 were affected by this bug, so I'd say only these below need a relaunch 😅

fgalan commented 1 year ago

I have relaunched compilation in all ones in the above list. However, given that sometimes it has been concurrently and I'm sure what RTD does in that case, could you have a look to all them (checking the ones that are already fixed), please? Thanks!

LZimmermannLZI commented 1 year ago

I checked the above versions to include the warning message and it is present in all entries:

Unfortunately the link in the warning note is not working properly. Clicking the link in a subfolder of the documentation provokes a 404 error. The reason seems to be that the link only changes the version number in the URL. The URL structure of the docs seem to have changed in the newest version. Example: The actual URL for the pagination subtopic of v 3.7.0 looks like this: https://fiware-orion.readthedocs.io/en/3.7.0/user/pagination.html

The redirected link of the warning note of v 1.15.0 looks like this: https://fiware-orion.readthedocs.io/en/3.7.0/user/pagination/index.html

The Problem doesnt occur at the starting page of each version doc.

fgalan commented 1 year ago

Thanks for your review!

Unfortunately the link in the warning note is not working properly. Clicking the link in a subfolder of the documentation provokes a 404 error. The reason seems to be that the link only changes the version number in the URL. The URL structure of the docs seem to have changed in the newest version. Example: The actual URL for the pagination subtopic of v 3.7.0 looks like this: https://fiware-orion.readthedocs.io/en/3.7.0/user/pagination.html

The redirected link of the warning note of v 1.15.0 looks like this: https://fiware-orion.readthedocs.io/en/3.7.0/user/pagination/index.html

The Problem doesnt occur at the starting page of each version doc.

Not sure how to fix this. It seems to be a RTD thing, isn't it?

Maybe you could open an issue on RTD and include the link as comment here (for traceability), as you did before, please? Thanks!

Maghnie commented 1 year ago

As suggested in readthedocs/readthedocs.org#9644, the page redirects approach could be used to correctly reroute older subpages of the documentation.

Maybe something like this can be added to the readthedocs configuration:

Type: Page Redirect
From URL: /multitenancy/index.html
To URL: /multitenancy.html

With some kind of regex in place of multitenancy.

fgalan commented 1 year ago

Maybe you could do a pull request on Orion repository to show how it looks like, please? Thanks!

Maghnie commented 1 year ago

Maybe we can, but this issue is gonna need the "backlog" label 😅

fgalan commented 1 year ago

Maybe we can, but this issue is gonna need the "backlog" label 😅

No problem. "backlog" label assigned ;)