Add second generic to UIMatch for handle field

[brophdawg11]

Add second handle generic to UIMatch to match the React Router version

Closes: #7438

[changeset-bot]

🦋 Changeset detected

Latest commit: 6d44e50

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/react	Patch
@remix-run/testing	Patch
create-remix	Patch
remix	Patch
@remix-run/architect	Patch
@remix-run/cloudflare	Patch
@remix-run/cloudflare-pages	Patch
@remix-run/cloudflare-workers	Patch
@remix-run/css-bundle	Patch
@remix-run/deno	Patch
@remix-run/dev	Patch
@remix-run/eslint-config	Patch
@remix-run/express	Patch
@remix-run/node	Patch
@remix-run/serve	Patch
@remix-run/server-runtime	Patch

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

[github-actions]

🤖 Hello there,

We just published version v0.0.0-nightly-fccadba-20230919 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[github-actions]

🤖 Hello there,

We just published version 2.0.1-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[github-actions]

🤖 Hello there,

We just published version 2.0.1 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[lensbart]

I just tried it out in v2.0.1, but it doesn’t work entirely satisfactory with things like remix-typedjson.

Since this is typed as export type UIMatch<D = AppData, H = RouteHandle> = UIMatchRR<SerializeFrom<D>, H>, it imposes SerializeFrom whereas people using remix-typedjson would want to use RemixSerializedType instead.

Would it be possible to just reexport UIMatch from @remix-run/router instead?

Thanks!

[kiliman]

Since remix-typedjson changes the data that is returned, in addition to the types, it's possible some of the utility functions/types will need to be added to my package.

[brophdawg11]

Would it be possible to just reexport UIMatch from @remix-run/router instead?

If you need that to bypass the Remix typings - you can import that directly from @remix-run/router, but I believe the current Remix type is correct for the majority of Remix use cases.

[lensbart]

@brophdawg11 that works for me. Thanks for your advice. (Still, I wonder if my use case (using remix-typedjson and typing the export const handle using the loader type) is exceptional enough to be okay with these ergonomics. Either way, I’m fine with it, just some food for thought.)

See also #7518 (comment)

[kiliman]

@lensbart can you add an issue in remix-typedjson with an example that's failing?

Thanks!