Skip to content
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

Linkage between admiral sites is broken #131

Closed
bms63 opened this issue Jul 24, 2023 · 5 comments · Fixed by #141
Closed

Linkage between admiral sites is broken #131

bms63 opened this issue Jul 24, 2023 · 5 comments · Fixed by #141
Assignees

Comments

@bms63
Copy link
Collaborator

bms63 commented Jul 24, 2023

@cicdguy @galachad @dgrassellyb

Hi all,

We noticed that the links between admiral packages are broken since we moved to the versioning system of the website.

For example, in the vignette derive_vars_merged() links out to a fallback page at https://rdrr.io/cran/admiral/man/derive_vars_merged.html, but we would like it to go to the admiral page for derive_vars_merged()

image

@cicdguy
Copy link
Collaborator

cicdguy commented Jul 24, 2023

Not controlled by us. It's part of pkgdown. See here for related issues: https://github.com/r-lib/pkgdown/issues?q=is%3Aissue+rdrr.io+is%3Aclosed

@ddsjoberg
Copy link

ddsjoberg commented Jul 24, 2023

Thanks for the link @cicdguy ! While I still need to review the linked issues, my first guess is the auto-linking broke when the main site was moved to a subfolder within the gh-pages branch. (By default, pkgdown will place the static website files in the root of gh-pages.)

Related and discussed today in the admiral meeting today:

  • move the primary website the default pkgdown location, aka the gh-pages root.

  • use the pkgdown built-in infrastructure for serving the dev site from a dev folder

    development:
      mode: auto
      version_label: default
  • The version menu on the website will only continue to grow. Proposal to reduce the list to "Development", "Current Release", and "Prior Releases". The "Prior Releases" would link to a separate page that would then list every prior release's website. My preference would be to put these release websites in a subfolder gh-pages/prior-releases. There is currently one folder for each release website in the root of the branch.

  • We can likely remove the admiral/docs/pkgdown.yml file from the main repo.

@galachad
Copy link
Member

galachad commented Aug 9, 2023

Hi @bms63 and @ddsjoberg I made the links tweak that will fix the issue with linking package to pharmaverse docs. pharmaverse/admiralvaccine#221 we can make it global in admiralci or have the tweaks in every package separately.

@bms63
Copy link
Collaborator Author

bms63 commented Aug 9, 2023

global sounds easier to maintain consistency.

@ddsjoberg
Copy link

Very clever use of .onLoad()!

Before this goes out to all admiral pkgs, I wonder if we should see if the move to GitFlow and placing the primary website in the root of the gh-pages branch will fix the issue (then we wouldn't need to maintain the fix into the future).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

4 participants