Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Conversation

@joadoumie
Copy link
Contributor

@joadoumie joadoumie commented Sep 5, 2023

Summary of the pull request

Files of type "*.winget" can launch Dev Home into the machine configuration flow.

Detailed description of the pull request / Additional comments

This PR adds logic into Dev Home to add ".winget" files as an activation handler so that users can launch directly into the Dev Home machine configuration flow by opening a ".winget" file with Dev Home.

Validation steps performed

  • Tested launching valid .winget file while Dev Home was closed, and while Dev Home was already open.
  • Tested launching invalid .winget file while Dev Home was closed, and while Dev Home was already open.
  • Tested launching a file while already in the machine config flow with a different file.
  • Tested launching a file while already in the machine config flow with the existing file.

PR checklist

@joadoumie
Copy link
Contributor Author

@guimafelipe - as discussed there is a bug where the flow doesn't work for an invalid .dsc file

@joadoumie
Copy link
Contributor Author

Fixed bug for invalid .dsc file

…n. Need to wait for the shell service to finish running on launch before handling file activation
@bbonaby
Copy link
Contributor

bbonaby commented Sep 27, 2023

@joadoumie I noticed this has still been in draft for about 2 weeks now. Do you still need a review on this or are you still working on it?

@joadoumie
Copy link
Contributor Author

@bbonaby I was OOF for a while... definitely do want to get this thing reviewed and make the necessary changes to get it merged in.

@joadoumie joadoumie marked this pull request as ready for review October 20, 2023 13:35
@joadoumie
Copy link
Contributor Author

Following up on this.

@denelon for visibility

@joadoumie joadoumie requested review from bbonaby and dkbennett and removed request for dkbennett October 25, 2023 16:01
@EricJohnson327
Copy link
Contributor

Ping @joadoumie Is this still wanted? Converting to draft until comments are taken care of.

@EricJohnson327 EricJohnson327 marked this pull request as draft January 11, 2024 20:29
@joadoumie
Copy link
Contributor Author

@EricJohnson327 yes definitely we still want it!

@AmelBawa-msft was making a bunch of changes though that will impact this flow so I was waiting until those PRs are merged in to work with Amir to add the functionality back in.

@AmelBawa-msft
Copy link
Contributor

Closing in favor of #2379

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.

Ability to open a YAML file directly into the machine configuration yaml flow

4 participants