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

feat(react,server-runtime): Support for v2 array syntax for route meta #4610

Merged
merged 10 commits into from
Nov 19, 2022

Conversation

chaance
Copy link
Collaborator

@chaance chaance commented Nov 15, 2022

This is the implementation for #4462.

Closes: #2598

Note: The types here are not complete and not ideal, but we are running into a wall with our current approach of copy-paste-all-the-types because we lack a shared dependency, and continuing down that path will make it even harder to fix. We are prioritizing a solution to that in a separate proposal and providing minimal (but sufficient) types in the mean time.

Also, docs will come in a separate PR because I'd like us to kick the tires on it a bit before documenting the change. This is fine since it's completely opt-in anyway.

  • Docs
  • Tests

@changeset-bot
Copy link

changeset-bot bot commented Nov 15, 2022

🦋 Changeset detected

Latest commit: 4b84867

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages
Name Type
@remix-run/dev Major
@remix-run/react Major
@remix-run/server-runtime Major
create-remix Major
@remix-run/cloudflare Major
@remix-run/deno Major
@remix-run/node Major
@remix-run/cloudflare-pages Major
@remix-run/cloudflare-workers Major
@remix-run/architect Major
@remix-run/express Major
@remix-run/netlify Major
@remix-run/vercel Major
@remix-run/serve Major
remix Major
@remix-run/eslint-config Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@chaance chaance marked this pull request as ready for review November 17, 2022 00:16
integration/meta-test.ts Outdated Show resolved Hide resolved
@chaance chaance merged commit 3f878a8 into dev Nov 19, 2022
@chaance chaance deleted the chance/future-meta branch November 19, 2022 00:31
@github-actions github-actions bot added the awaiting release This issue has been fixed and will be released soon label Nov 19, 2022
kentcdodds pushed a commit that referenced this pull request Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting release This issue has been fixed and will be released soon CLA Signed package:server-runtime renderer:react
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants