-
Notifications
You must be signed in to change notification settings - Fork 4
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
First GPOs guide draft #81
Conversation
GabrielaReyna
commented
May 7, 2024
•
edited by bates64
edited by bates64
- Fix Change /cheerpj-jnlprunner/ links to /cheerpj-jnlp-runner/ #80
Deploying labs-leaningtech-com with Cloudflare Pages
|
Tests are failing because of #80 |
src/content/docs/cheerpj-jnlp-runner/04-guides/private-hosting-chrome.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/cheerpj-jnlp-runner/04-guides/private-hosting-chrome.mdx
Show resolved
Hide resolved
src/content/docs/cheerpj-jnlp-runner/04-guides/private-hosting-chrome.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/cheerpj-jnlp-runner/04-guides/private-hosting-chrome.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/cheerpj-jnlp-runner/04-guides/private-hosting-chrome.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/cheerpj-jnlp-runner/04-guides/private-hosting-chrome.mdx
Outdated
Show resolved
Hide resolved
I'm getting broken link errors, it is trying to map to the older cheerpj-jnlprunner url. Any ideas on how to fix this? |
Yep, this is a known issue #80 |
Commit 94d45ec could have "fix #80" in the commit message so GitHub knows to close the issue when it is merged. See https://docs.github.com/en/issues/tracking-your-work-with-issues/closing-an-issue. I'll edit the PR description instead in this instance |
Do we prefer that Stefano reviews this? If my changes were resolved I'm happy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, assigning @sderossi for second content review