Skip to content

Conversation

@mattleibow
Copy link
Member

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description of Change

Now that the _framework/hybridwebview.js file is embedded, we can make it available for consumption by the app. It is not required to use this URL, but it is now available.

This being embedded makes things easier to update and keep in sync with the implementation of the HybridWebView.

Issues Fixed

Fixes #29788

@mattleibow mattleibow requested a review from a team as a code owner June 3, 2025 01:40
@mattleibow mattleibow requested review from PureWeen and jfversluis and removed request for a team June 3, 2025 01:40
@mattleibow mattleibow changed the base branch from dev/hybridwebview-backport-ts to net10.0 June 3, 2025 01:40
@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Now that the _framework/hybridwebview.js file is embedded, we can
make it available for consumption by the app. It is not required
to use this URL, but it is now available.

This being embedded makes things easier to update and keep in sync
with the implementation of the HybridWebView.
@mattleibow mattleibow force-pushed the dev/hybridwebview-framework-js branch from 6efcfe6 to ac09178 Compare June 3, 2025 11:57
@rmarinho rmarinho merged commit f6079e7 into net10.0 Jun 4, 2025
129 checks passed
@rmarinho rmarinho deleted the dev/hybridwebview-framework-js branch June 4, 2025 13:13
@mattleibow
Copy link
Member Author

/backport to release/10.0.1xx-preview5

@github-actions
Copy link
Contributor

github-actions bot commented Jun 4, 2025

Started backporting to release/10.0.1xx-preview5: https://github.com/dotnet/maui/actions/runs/15444004043

@mattleibow
Copy link
Member Author

/backport to main

@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

@mattleibow backporting to "main" failed, the patch most likely resulted in conflicts:

$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Applying: Expose _framework/hybridwebview.js
Using index info to reconstruct a base tree...
M	src/Controls/samples/Controls.Sample/Maui.Controls.Sample.csproj
M	src/Controls/tests/DeviceTests/Controls.DeviceTests.csproj
M	src/Controls/tests/DeviceTests/Resources/Raw/HybridTestRoot/index.html
M	src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.Windows.cs
M	src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.cs
M	src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.iOS.cs
M	src/Core/src/Platform/Android/MauiHybridWebViewClient.cs
Falling back to patching base and 3-way merge...
Auto-merging src/Core/src/Platform/Android/MauiHybridWebViewClient.cs
Auto-merging src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.iOS.cs
CONFLICT (content): Merge conflict in src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.iOS.cs
Auto-merging src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.cs
Auto-merging src/Core/src/Handlers/HybridWebView/HybridWebViewHandler.Windows.cs
Auto-merging src/Controls/tests/DeviceTests/Resources/Raw/HybridTestRoot/index.html
Auto-merging src/Controls/tests/DeviceTests/Controls.DeviceTests.csproj
Auto-merging src/Controls/samples/Controls.Sample/Maui.Controls.Sample.csproj
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Expose _framework/hybridwebview.js
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants