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

[BUG] Fix broken docs link for EuiProvider props. #7273

Merged
merged 3 commits into from
Oct 10, 2023

Conversation

1Copenut
Copy link
Contributor

@1Copenut 1Copenut commented Oct 10, 2023

Summary

I found a broken link that 404's that was tangential to #7272. This PR fixes that broken link so it points to the correct EuiProvider props anchor.

QA

QA will be done manually in the PR preview page for #/utilities/provider#cache-location

General checklist

  • Browser QA
    • Checked in both light and dark modes
    • Checked in mobile
    • Checked in Chrome, Safari, Edge, and Firefox

@1Copenut 1Copenut added bug documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog labels Oct 10, 2023
@1Copenut 1Copenut self-assigned this Oct 10, 2023
@1Copenut 1Copenut requested a review from a team as a code owner October 10, 2023 15:48
@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @1Copenut

Copy link
Member

@cee-chen cee-chen left a comment

Choose a reason for hiding this comment

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

🎉 Thanks for fixing this Trevor!

@1Copenut 1Copenut merged commit 36bd3ee into elastic:main Oct 10, 2023
7 checks passed
@1Copenut 1Copenut deleted the bug/eui-provider-props-link branch October 10, 2023 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants