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.

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

feat(microsoft_store): Add Microsoft Store step for Windows #963

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

Andreas02-dev
Copy link
Contributor

@Andreas02-dev Andreas02-dev commented Oct 18, 2024

Add Microsoft Store Apps update step for Windows as Winget cannot update all Microsoft Store apps yet.

Closes #912.

What does this PR do

Adds a Microsoft Store Apps update step for Windows.

Standards checklist

  • The PR title is descriptive.
  • I have read CONTRIBUTING.md
  • Optional: I have tested the code myself

For new steps

  • Optional: Topgrade skips this step where needed
  • Optional: The --dry-run option works with this step
  • Optional: The --yes option works with this step if it is supported by
    the underlying command

If you developed a feature or a bug fix for someone else and you do not have the
means to test it, please tag this person here.

@Rikiub, could you verify the Spanish translations?
@olivertzeng, could you verify the Taiwanese translations?

Success

Success

Error

Error
Note that the '0' in output will be a number other than 0 when a real error occurs.

Copy link
Contributor

@Rikiub Rikiub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spanish translations looks great! Nothing to change.

Copy link
Member

@SteveLauC SteveLauC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the implementation!

locales/app.yml Outdated
"Microsoft Store":
en: "Microsoft Store"
es: "Tienda de Microsoft"
zh_TW: "Microsoft 店鋪"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
zh_TW: "Microsoft 店鋪"
zh_TW: "Microsoft Store"

Looks like "Microsoft Store" won't be translated:

text in the below picture: Microsoft Store Support

Screenshot 2024-10-22 at 9 35 43 PM

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true
圖片

@SteveLauC
Copy link
Member

Just checked the zh_TW translations and left a comment, let's merge the PR once we have it resolved:)

BTW, there is a conflict in locales/app.yml, branch needs a rebase.

Add Microsoft Store Apps update step for Windows as Winget cannot update all Microsoft Store apps yet.

Closes topgrade-rs#912
@Andreas02-dev
Copy link
Contributor Author

Thanks @SteveLauC @olivertzeng, should be ready to merge now then!

Copy link
Member

@SteveLauC SteveLauC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@SteveLauC SteveLauC merged commit ea2f3e0 into topgrade-rs:main Oct 23, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature | Microsoft Store app update support
4 participants