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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update with-prefetching example #9101

Merged
merged 2 commits into from
Oct 18, 2019
Merged

Update with-prefetching example #9101

merged 2 commits into from
Oct 18, 2019

Conversation

HaNdTriX
Copy link
Contributor

Changelog

  • Remove getInitialProps
  • Explain variants
  • Reflect latest changes in the prefetch api
  • Update UI

@HaNdTriX HaNdTriX requested review from lfades and Timer as code owners October 16, 2019 13:20
@HaNdTriX
Update with-prefetching example
9e76360 
## Changelog

- Remove `getInitialProps`
- Explain variants
- Reflect latest changes in the prefetch api
- Update UI
@HaNdTriX HaNdTriX force-pushed the examples/with-prefetching branch from 48d250d to 9e76360 Compare October 16, 2019 13:21
@Timer Timer added ready to land examples Issue was opened via the examples template. labels Oct 18, 2019
Timer
Timer approved these changes Oct 18, 2019
View reviewed changes
Copy link
Member

@Timer Timer left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@Timer Timer added this to the 9.1.2 milestone Oct 18, 2019
@Timer Timer merged commit 16e0ff9 into vercel:canary Oct 18, 2019
@HaNdTriX HaNdTriX deleted the examples/with-prefetching branch October 18, 2019 12:22
This was referenced Nov 12, 2019
Merged
Open
Open
Open
Merged
@snyk-bot snyk-bot mentioned this pull request Nov 25, 2019
Open
@vercel vercel locked as resolved and limited conversation to collaborators Feb 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Timer Timer Timer approved these changes

@lfades lfades Awaiting requested review from lfades

Assignees
No one assigned
Labels
examples Issue was opened via the examples template.
Projects
None yet
Milestone
9.1.2
Development

Successfully merging this pull request may close these issues.
2 participants
@HaNdTriX @Timer