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

Prerender all routes nested in Set with prerender prop #2542

Merged
merged 4 commits into from May 18, 2021

Conversation

Tobbe
Copy link
Member

@Tobbe Tobbe commented May 16, 2021

This PR adds support for specifying prerender on <Set>s. All routes in a Set like that will be prerendered.

<Set prerender>
  <Route path="/foo" page={FooPage} name="fooPage" />
  <Route path="/bar" page={BarPage} name="barPage" />
</Set>

is equivalent to

<Route path="/foo" page={FooPage} name="fooPage" prerender />
<Route path="/bar" page={BarPage} name="barPage" prerender />

@Tobbe
Copy link
Member Author

Tobbe commented May 16, 2021

@dac09
Copy link
Collaborator

dac09 commented May 17, 2021

LGTM minus failing test!

@jtoar jtoar added this to In progress in Router via automation May 18, 2021
@Tobbe
Copy link
Member Author

Tobbe commented May 18, 2021

@dac09 Ready for your ✔️

@peterp peterp added this to the next-release milestone May 18, 2021
@dac09 dac09 merged commit a1d8722 into redwoodjs:main May 18, 2021
Router automation moved this from In progress to Done May 18, 2021
@Tobbe Tobbe deleted the tobbe-set-prerender branch May 18, 2021 11:24
dac09 added a commit to dac09/redwood that referenced this pull request May 19, 2021
…-codegen

* 'main' of github.com:redwoodjs/redwood: (54 commits)
  Add private methods loose explicitly (redwoodjs#2554)
  Custom `useAuth` pass through for `RedwoodApolloProvider` (redwoodjs#2455)
  Prerender all routes nested in Set with prerender prop (redwoodjs#2542)
  Upgrade eslint and prettier packages including formatting fixes (redwoodjs#2540)
  contributors updates (redwoodjs#2544)
  Rename default datasource (redwoodjs#1941)
  Add default config for Component generation (redwoodjs#1814)
  build(deps): bump core-js from 3.10.1 to 3.12.1 (redwoodjs#2481)
  upgrade babel 7.14.2 with misc babel packages (redwoodjs#2541)
  build(deps): bump http-proxy-middleware from 1.1.0 to 2.0.0 (redwoodjs#2536)
  build(deps): bump pino-pretty from 4.7.1 to 4.8.0 (redwoodjs#2534)
  build(deps): bump concurrently from 6.0.2 to 6.1.0 (redwoodjs#2533)
  build(deps-dev): bump firebase-admin from 9.7.0 to 9.8.0 (redwoodjs#2522)
  build(deps): bump esbuild-loader from 2.10.0 to 2.13.0 (redwoodjs#2518)
  build(deps): bump @graphql-tools/merge from 6.2.13 to 6.2.14 (redwoodjs#2516)
  updating minor dependency versions across packages (redwoodjs#2532)
  Add JSON headers to Function generator template (redwoodjs#2457)
  fixed firebase promises so that they can be caught (redwoodjs#2503)
  Prevent re-initialization of a Firebase app that is already initialized (redwoodjs#2504)
  build(deps-dev): bump magic-sdk from 2.7.0 to 4.3.0 (redwoodjs#2463)
  ...
@thedavidprice
Copy link
Contributor

@dac09 should Prerender (and/or Sets) docs be updated?

@dac09
Copy link
Collaborator

dac09 commented May 22, 2021

yep added label :)

@Tobbe
Copy link
Member Author

Tobbe commented May 23, 2021

@thedavidprice @dac09 I already did the docs. Linked above.

redwoodjs/redwoodjs.com#697

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Router
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants