Skip to content
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

[UI] [WinGet] Sync package selection on UI and install packages #112

Merged
merged 18 commits into from
Mar 28, 2023
Merged

[UI] [WinGet] Sync package selection on UI and install packages #112

merged 18 commits into from
Mar 28, 2023

Conversation

AmelBawa-msft
Copy link
Contributor

🚀 What was done?

  • Added install functionality
  • Synchronize package selection on the UI (GIF below):
    • If a package appears on popular apps, and restore apps and in the search result of several queries, ensure that if that package was selected, the UI reflects the change everywhere the package surfaces.
  • Added Unit test

🧪 Test?

Synchronize selection

output2

Install package

@AmelBawa-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

florelis
florelis reviewed Mar 27, 2023
View reviewed changes
@AmelBawa-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AmelBawa-msft AmelBawa-msft merged commit 4bd8f82 into main Mar 28, 2023
@AmelBawa-msft AmelBawa-msft deleted the user/amelbawa-msft/install branch March 28, 2023 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@florelis florelis florelis approved these changes

@dhoehna dhoehna Awaiting requested review from dhoehna

@asklar asklar Awaiting requested review from asklar

@bbonaby bbonaby Awaiting requested review from bbonaby

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AmelBawa-msft @florelis