Skip to content

Conversation

@Kuinox
Copy link
Contributor

@Kuinox Kuinox commented Nov 9, 2022

Fixed a concatenation issue in special URIs.

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Description

When the URL end with an hash, current code concatenate the file name to the URL, which doesn't lead in the wanted behavior.

Fixes #44927.

@Kuinox Kuinox requested a review from a team as a code owner November 9, 2022 12:45
@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Nov 9, 2022
@ghost
Copy link

ghost commented Nov 9, 2022

Thanks for your PR, @Kuinox. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@mkArtakMSFT mkArtakMSFT added the area-blazor Includes: Blazor, Razor Components label Nov 9, 2022
Copy link
Member

@MackinnonBuck MackinnonBuck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution, @Kuinox!

@MackinnonBuck MackinnonBuck merged commit bb6a05b into dotnet:main Nov 9, 2022
@ghost ghost added this to the 8.0-preview1 milestone Nov 9, 2022
@Kuinox Kuinox deleted the patch-1 branch November 9, 2022 23:26
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 community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Blazor JS Interop is bugged in an hard to reproduce scenario.

3 participants