Skip to content

[7.0] Finalize JSImport/JSExport interop#27385

Merged
guardrex merged 3 commits into
mainfrom
guardrex/blazor-js-interop
Oct 26, 2022
Merged

[7.0] Finalize JSImport/JSExport interop#27385
guardrex merged 3 commits into
mainfrom
guardrex/blazor-js-interop

Conversation

@guardrex
Copy link
Copy Markdown
Collaborator

@guardrex guardrex commented Oct 26, 2022

Addresses #27016
Addresses #26364

Pavel ... I assume that we can place a copy of the .NET-JavaScript type mappings table in the Blazor JS interop article. I use a versioned INCLUDE file for it to avoid duplicated content. The file is also versioned, so it can be easily updated release-to-release.

btw - I noticed a NIT update 😈 to make for another INCLUDE file (size-limits.md). I forgot to place a 7.0 version of that file for some earlier updates. I'm just going to tack that on here to knock it out 🥊😵.

@guardrex guardrex requested a review from pavelsavara October 26, 2022 11:15
@guardrex guardrex mentioned this pull request Oct 26, 2022
8 tasks
@guardrex
Copy link
Copy Markdown
Collaborator Author

guardrex commented Oct 26, 2022

Note that because this touches the What's New and Migration articles that I can only leave it up for a few hours. We're trying not to step on each other's PRs that touch these topics. If you don't get a chance to review before I merge, you can post feedback here for a patch update later.

@guardrex guardrex merged commit d96bcca into main Oct 26, 2022
@guardrex guardrex deleted the guardrex/blazor-js-interop branch October 26, 2022 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant