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

Broken link to record-deployment #92

Closed
TimothyJones opened this issue Aug 12, 2021 · 3 comments
Closed

Broken link to record-deployment #92

TimothyJones opened this issue Aug 12, 2021 · 3 comments

Comments

@TimothyJones
Copy link
Contributor

Dmitry Munda pointed out on slack that on this page:

https://docs.pact.io/pact_broker/client_cli/readme#deployments

There is this link:

https://docs.pact.io/record-deployment

which is broken and should be:

https://docs.pact.io/pact_broker/recording_deployments_and_releases/

However, when opening the readme in this repo to fix it, I saw that it's inside the autogenerated docs section. I'm not sure how that works, so I'm opening an issue instead.

@bethesque
Copy link
Member

Thanks. I needed to add the short link redirects to the netlify config pact-foundation/docs.pact.io@a0782fe

I do this so I can put short links in the output of the CLI tools.

@TimothyJones
Copy link
Contributor Author

TimothyJones commented Aug 13, 2021 via email

@bethesque
Copy link
Member

If it's something you want to print to the screen, put a "/go/" in the link, and then in the forwarding, put in the tracking information in the redirect URL. You'll see it in the netlify file. We're using this to work out if the links we put in the output are actually effective.

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

No branches or pull requests

2 participants