Skip to content

[browser] Fix asset loading in boot config scenario #86240

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
maraf merged 1 commit into from
May 15, 2023
Merged

[browser] Fix asset loading in boot config scenario #86240

maraf merged 1 commit into from
May 15, 2023

Conversation

@maraf
Copy link
Member

@maraf maraf commented May 15, 2023

@maraf
- Fix using downloadResource
b217c62 
- Fix loading boot config with startupOptions.loadBootResource
@maraf maraf added this to the 8.0.0 milestone May 15, 2023
@maraf maraf self-assigned this May 15, 2023
@maraf maraf requested review from kg, lewing and pavelsavara as code owners May 15, 2023 09:58
@ghost
Copy link

ghost commented May 15, 2023

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details
Author: maraf
Assignees: maraf
Labels:

arch-wasm, area-System.Runtime.InteropServices.JavaScript
Milestone: 8.0.0

@maraf maraf merged commit d6797a5 into dotnet:main May 15, 2023
@maraf maraf deleted the WasmFixConfigLoading branch May 15, 2023 12:19
@build-analysis build-analysis bot mentioned this pull request May 15, 2023
Closed
@ghost ghost locked as resolved and limited conversation to collaborators Jun 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@pavelsavara pavelsavara pavelsavara approved these changes

@lewing lewing Awaiting requested review from lewing lewing is a code owner

@kg kg Awaiting requested review from kg

Assignees

@maraf maraf

Labels

arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript

Projects

None yet

Milestone

8.0.0

Development

Successfully merging this pull request may close these issues.

2 participants

@maraf @pavelsavara