Skip to content

Conversation

oroztocil
Copy link
Member

Adds default implementations for new interface and abstract methods that were added for the Blazor JS interop in #61246.

Removes the breaking change described in #61842.

@oroztocil oroztocil requested a review from a team as a code owner May 9, 2025 10:59
@github-actions github-actions bot added the area-blazor Includes: Blazor, Razor Components label May 9, 2025
@oroztocil oroztocil changed the title Add default implementations for new interface methods in Blazor JS in… Add default implementations for new interface methods in Blazor JS interop May 9, 2025
@oroztocil
Copy link
Member Author

Should there be a custom message in the NotImplementedExceptions?

@javiercn
Copy link
Member

javiercn commented May 9, 2025

Should there be a custom message in the NotImplementedExceptions?

Look for the exception and do the same thing that we do in other cases

@oroztocil
Copy link
Member Author

Should there be a custom message in the NotImplementedExceptions?

Look for the exception and do the same thing that we do in other cases

Yes, did that and usually there is none.

@oroztocil oroztocil enabled auto-merge (squash) May 9, 2025 11:17
@oroztocil oroztocil merged commit c3fb958 into main May 12, 2025
26 of 27 checks passed
@oroztocil oroztocil deleted the oroztocil/js-interop-default-interface-implementations branch May 12, 2025 16:53
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview5 milestone May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Includes: Blazor, Razor Components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants