Skip to content

Blazor Desktop cleanup tasks for preview 3 #30656

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
Closed
10 of 13 tasks
Eilon opened this issue Mar 4, 2021 · 2 comments
Closed
10 of 13 tasks

Blazor Desktop cleanup tasks for preview 3 #30656

Eilon opened this issue Mar 4, 2021 · 2 comments
Assignees
@javiercn
Labels
area-blazor Includes: Blazor, Razor Components Done This issue has been fixed feature-blazor-desktop This issue is related to Blazor Desktop task
Milestone
6.0-preview3

Comments

@Eilon
Copy link
Contributor

Eilon commented Mar 4, 2021 •
edited by mkArtakMSFT
Loading

  • Copyright headers - Eilon
  • Review general public API surface - Eilon
  • XML docs - Eilon
  • Exclude WPF/WinForms build from non-Windows platforms. See if what we have just works already. If not, talk to Will/Doug - Javier
  • Public API signatures test/TXT file - Javier
  • WebView2 NuGet failure - Eilon
  • Review TODO comments - Steve
  • Runs on CI - everyone keep an eye, but post a message here if you're looking at something so we don't overlap
  • Make sure the right packages are actually marked as shipping and show up in nightly feeds (at minimum, this means Microsoft.AspNetCore.Components.WebView - the WinForms/WPF ones are TBD) - Javier once plan confirmed
  • Manual test/verification pass. Can we make BasicTestApp run inside the webview and manually click through all the scenarios? What about a bigger app like CarChecker? - Steve
  • Some kind of getting started content - Eilon (will do this post-Preview 3 when we have .NET MAUI scenarios working)
    • Written guide (maybe blog post, maybe docs)
    • Official sample projects (maybe on asplabs)
@Eilon Eilon added the feature-blazor-desktop This issue is related to Blazor Desktop label Mar 4, 2021
@Pilchie Pilchie added the area-blazor Includes: Blazor, Razor Components label Mar 4, 2021
@SteveSandersonMS SteveSandersonMS mentioned this issue Mar 8, 2021
Merged
@SteveSandersonMS SteveSandersonMS removed their assignment Mar 8, 2021
@mkArtakMSFT mkArtakMSFT added this to the 6.0-preview3 milestone Mar 8, 2021
@SteveSandersonMS SteveSandersonMS mentioned this issue Mar 18, 2021
Merged
@Eilon
Copy link
Contributor Author

Eilon commented Mar 18, 2021

Public API surface was already reviewed and quite minimal to begin with, so that's done for now.

@Eilon
Copy link
Contributor Author

Eilon commented Mar 18, 2021

@javiercn - once you're done marking the packages as shipping (WPF/WinForms and also WebView2), go ahead and close this as done!

@Eilon Eilon removed their assignment Mar 18, 2021
@ghost ghost added Done This issue has been fixed and removed Working labels Mar 22, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Apr 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@javiercn javiercn

Labels
area-blazor Includes: Blazor, Razor Components Done This issue has been fixed feature-blazor-desktop This issue is related to Blazor Desktop task
Projects
None yet
Milestone
6.0-preview3
Development

No branches or pull requests
5 participants
@Eilon @Pilchie @SteveSandersonMS @javiercn @mkArtakMSFT