Docs preview failed credentials

[spossann]

There are problems with launching the docs preview workflow, manually:

https://github.com/struphy-hub/struphy/actions/runs/29913011771/job/88900201061

and after merge:

https://github.com/struphy-hub/struphy/actions/runs/29912433362/job/88898269509

I gave the struphy-dev team write access to the repo, but it didn't change:

https://github.com/struphy-hub/docs-preview/settings/access

Maybe the access token should go into the organization, like the GHCR_PROJECT_TOKEN:

https://github.com/organizations/struphy-hub/settings/secrets/actions

[max-models]

Sorry, I accidentally deleted the credentials today

[max-models]

@spossann can you try again now? Seems to work: https://github.com/struphy-hub/struphy/actions/runs/29913011771/job/88940515182

[spossann]

The run finishes now. However, the site is not displayed: https://struphy-hub.github.io/docs-preview/

Maybe the path for index.html must be in / [root] or /docs.

[max-models]

The run finishes now. However, the site is not displayed: https://struphy-hub.github.io/docs-preview/

Maybe the path for index.html must be in / [root] or /docs.

This is the correct link: https://struphy-hub.github.io/docs-preview/pr-284/, we can add a "Print link" stage so it's more clear where to find it, otherwise the github bot posts it in the PR like here #283 (comment)

[spossann]

Is it possoble that the links show up under deployments?

https://github.com/struphy-hub/docs-preview/deployments

[max-models]

Is it possoble that the links show up under deployments?

https://github.com/struphy-hub/docs-preview/deployments

I'm not 100% sure where you want the link to appear, but it might be possible by changing the environment variable like I did in #289, can you try it out?

[spossann]

Where does the link https://struphy-hub.github.io/docs-preview/pr-284/ show up at the moment?

[spossann]

It seems that older links are not persistent:

https://struphy-hub.github.io/docs-preview/pr-283/

So why not just put the page under the default https://struphy-hub.github.io/docs-preview/ ?

[max-models]

1. The preview docs are cleaned up once a PR is merged into devel. When PR 283 was merged, the corresponding docs were removed from the preview repo (this breaks the link ofc).
2. The links show up as a comment in the PR iff the preview is deployed by adding [preview docs] to the commit message. See Added workflow to push docs to struphy-hub/docs-preview #283 (comment)
3. "why not just put the page under the default https://struphy-hub.github.io/docs-preview/", because sometimes (often) we have many open PRs and all of them should be able to preview the new docs at the same time. If we just use that link then it would overwrite the preview docs every time. If you want to, I can add a main page to the https://struphy-hub.github.io/docs-preview/ page from which you can get links to the individual previews.

[max-models]

Now https://struphy-hub.github.io/docs-preview/ is more meaningful, I added an action to the repo that updated the page on push to main. See here https://github.com/struphy-hub/docs-preview/blob/main/.github/workflows/build-index.yml