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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

<Link prefetch> #28

Closed
sedubois opened this issue Apr 27, 2017 · 3 comments
Closed

<Link prefetch> #28

sedubois opened this issue Apr 27, 2017 · 3 comments

Comments

@sedubois
Copy link

From reading the README, it's not clear if Link supports Next.js' prefetch option?

Thanks 馃檪
@HaNdTriX
Copy link
Collaborator

It does since it just extends the next/link link.

https://github.com/fridays/next-routes/blob/master/src/index.js#L54-L64

fridays added a commit that referenced this issue Apr 27, 2017
@fridays
Merge pull request #29 from HaNdTriX/master
00d7bd2 
Update Readme fix #28
@sedubois
Copy link
Author

OK, thanks! Don't know if documentation should reflect this?

@HaNdTriX
Copy link
Collaborator

Since you stumbled over it, I believe yes. Actually I also had to check the code to be sure.
So I chose a more general wording.

Other props like prefetch will work as well.

If next/link will get more props we don't need to update the docs.

SeanDemps pushed a commit to SeanDemps/next-routes that referenced this issue Dec 13, 2018
@elliottsj
Merge pull request fridays#28 from Yolk-HQ/renovate/node-10.x
365ad50 
Update dependency @types/node to v10.12.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@HaNdTriX @sedubois