Skip to content
This repository was archived by the owner on Jan 10, 2025. It is now read-only.

Fix @shopify/react-router <Link /> component#1073

Merged
cartogram merged 3 commits intomasterfrom
fix-link
Oct 2, 2019
Merged

Fix @shopify/react-router <Link /> component#1073
cartogram merged 3 commits intomasterfrom
fix-link

Conversation

@cartogram
Copy link
Contributor

@cartogram cartogram commented Oct 1, 2019

Description

This PR applies a small fix to the <Link /> that explicitly gives it a children prop. Without this change the component would get type errors if you just tried to render it like a normal component such as <Link url="">Some text</Link>.

Type of change

  • react-router Patch: Bug/ Documentation fix (non-breaking change which fixes an issue or adds documentation)

Checklist

  • I have added a changelog entry, prefixed by the type of change noted above

@cartogram cartogram changed the title Fix react-router -> Link component Fix @shopify/react-router <Link /> component Oct 1, 2019
@cartogram cartogram merged commit 058d3bd into master Oct 2, 2019
@cartogram cartogram deleted the fix-link branch October 2, 2019 03:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants