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

TuneUp - User facing strings moved to Resources #70

Merged
merged 8 commits into from
Oct 25, 2024

Conversation

ivaylo-matov
Copy link
Contributor

@ivaylo-matov ivaylo-matov commented Oct 14, 2024

PR to address a request from #66

Adds localization support. Hardcoded user-facing strings are extracted and moved to resources.

Also uses MessageBoxService.Show() to change the style or error message when ExportToCSV cannot overwrite a locked file.
This addresses a request in DYN-7436

@reddyashish
@QilongTang

@dnenov

resources created but TuneUp does not show in context menu
@ivaylo-matov
Copy link
Contributor Author

ivaylo-matov commented Oct 18, 2024

... Also added the Tooltips and ShowGroups is enabled by default.
Mural board

@QilongTang
Copy link
Contributor

Once resources are added, when you build TuneUp, do you get additional files other than just the dll? Can you give a screen shot

@ivaylo-matov
Copy link
Contributor Author

@QilongTang, after adding en-US, I now see a new folder with TuneUp.resources.dll created during the build.

@QilongTang
Copy link
Contributor

@ivaylo-matov Can you touch https://github.com/DynamoDS/TuneUp/blob/master/pipeline.yml#L31 to also include the new folder in the zip file?

@reddyashish reddyashish merged commit 41f5113 into DynamoDS:master Oct 25, 2024
10 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.

3 participants