Links in GitHub CI failures point to old RTD project URL

[stevepiercy]

Now that we have a beautiful new default UI for RTD at app.readthedocs.org, I noticed that when a pull request preview fails, then the links that are generated by RTD point to the old UI at readthedocs.org.

https://readthedocs.org/projects/volto/builds/26991012/

Should be:

https://app.readthedocs.org/projects/volto/builds/26991012/

Details

• Read the Docs project URL: https://readthedocs.org/projects/volto/
• Build URL (if applicable): https://readthedocs.org/projects/volto/builds/26991012/
• Read the Docs username (if applicable): https://readthedocs.org/profiles/stevepiercy/
• Example pull request with RTD build failure: Show working copy links based on backend actions plone/volto#6393

Expected Result

The link should point to the host app.readthedocs.org.

Actual Result

The link points to the host readthedocs.org.

---

As a side note, this issue template should also be updated to use the new URLs.

* Read the Docs project URL: https://app.readthedocs.org/projects/{your_project_slug}/
* Build URL (if applicable): https://app.readthedocs.org/projects/{your_project_slug}/builds/{build_id}/
* Read the Docs username (if applicable): https://app.readthedocs.org/profiles/{your_rtd_username}/

[humitos]

I tried fixing this some weeks ago and I got lost in the operation settings. They are pretty complex currently 🙃

This should be automatically fixed on Mar 11th, 2025, since we are removing the old dashboard, https://about.readthedocs.com/blog/2024/11/rollout-of-our-new-dashboard/

I'm adding it to Q3 so we can double check it's working after Mar 11th.

[flying-sheep]

This makes the banner a bit confusing: “you’re using the old dashboard, please switch to the new one” sounds like I’m supposed to do something to switch, while really there’s no action I can take to do that.

[humitos]

Yes, that's unfortunate. However, the work required to change this is big enough that we won't do it before March 11th, when the old dashboard is gone.

[humitos]

Hrm, I just checked this with #12206 and it seems the links still point to the old dashboard:

• https://readthedocs.org/dashboard/dev/version/12206/edit/
• https://readthedocs.org/dashboard/docs/version/12206/edit/

I will double check our settings to see if I can change it now.

I understand the code that generates this URL is https://github.com/readthedocs/readthedocs.org/blob/testing-pr/readthedocs/oauth/services/github.py#L484-L489

[humitos]

Hrm, yeah, for some reason the setting is invalid in web-celery instances:

docs@web-celery-i-08e116d41f8e8d692(org):~$ django-admin print_settings | grep PRODUCTION_DOMAIN
PRODUCTION_DOMAIN                        = 'readthedocs.org'
docs@web-celery-i-08e116d41f8e8d692(org):~$

[humitos]

I found the issue. We are still using readthedocs.settings.managed (PRODUCTION_DOMAIN = 'readthedocs.org') on web-celery but readthedocs.settings.exttheme on the rest of the instances.

This issue should be fixed tomorrow when we deploy the changes from https://github.com/readthedocs/readthedocs-ops/pull/1638

[humitos]

This is working now and those links should be pointing to the new dashboard 💯 . Let me know otherwise...