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

fix more doc links #7146

Merged
merged 1 commit into from
Oct 19, 2021
Merged

fix more doc links #7146

merged 1 commit into from
Oct 19, 2021

Conversation

jrhizor
Copy link
Contributor

@jrhizor jrhizor commented Oct 19, 2021

Doc check is failing on master for a variety of reasons. I think this catches them all.

@sherifnada I'm changing the Zuora endpoints to not be links since none of them are directly resolvable. Let me know if you want me to revert to ignoring them instead of just listing the domains.

@github-actions github-actions bot added the area/documentation Improvements or additions to documentation label Oct 19, 2021
@jrhizor jrhizor temporarily deployed to more-secrets October 19, 2021 01:10 Inactive
| US Production | [https://rest.zuora.com](https://rest.zuora.com) |
| US Cloud Production | [https://rest.na.zuora.com](https://rest.na.zuora.com) |
| EU Production | [https://rest.eu.zuora.com](https://rest.eu.zuora.com) |
| US Production | rest.zuora.com |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so does this effectively make the checker ignore them? if so, I would rather we keep the hyperlink. If this is not ignoring them then let's keep it this way.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checked in slack with @sherifnada, these are just api endpoint domains so we're fine dropping

@jrhizor jrhizor merged commit 2d32df1 into master Oct 19, 2021
@jrhizor jrhizor deleted the jrhizor/fix-more-doc-links branch October 19, 2021 01:21
schlattk pushed a commit to schlattk/airbyte that referenced this pull request Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants