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

Minor fixes and descriptive comments #2786

Merged
merged 2 commits into from
Sep 6, 2023
Merged

Conversation

skapur12
Copy link
Member

@skapur12 skapur12 commented Aug 24, 2023

  • Minor fixes/descriptive comments

Everything else (including the strings) has been moved to ClientResources in the portal PR: https://msazure.visualstudio.com/One/_git/Batch-UX-Portal/pullrequest/8654322

@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Merging #2786 (22fa2fa) into main (40e6500) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

❗ Current head 22fa2fa differs from pull request most recent head a3c54b9. Consider uploading reports for the commit a3c54b9 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2786   +/-   ##
=======================================
  Coverage   66.47%   66.47%           
=======================================
  Files        1226     1226           
  Lines       33705    33705           
  Branches     6150     6150           
=======================================
  Hits        22404    22404           
  Misses      11168    11168           
  Partials      133      133           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 054059b...a3c54b9. Read the comment docs.

@dpwatrous
Copy link
Member

There are some dependency ordering issues here. The bonito-* packages don't depend on any of the Batch packages, and shouldn't contain any Batch-specific strings. Can we move all of this logic to the portal's build process? Doing it in this repo seems odd.

@skapur12 skapur12 force-pushed the skapur/portal-translations branch 2 times, most recently from e7ab00f to a6003fb Compare August 29, 2023 00:39
@skapur12 skapur12 changed the title Compile package translations for use in the portal Add strings needed for portal localization Aug 29, 2023
@skapur12
Copy link
Member Author

There are some dependency ordering issues here. The bonito-* packages don't depend on any of the Batch packages, and shouldn't contain any Batch-specific strings. Can we move all of this logic to the portal's build process? Doing it in this repo seems odd.

The logic that compiles the shared libraries package translations together has now been moved out of this PR and into the portal PR: https://msazure.visualstudio.com/One/_git/Batch-UX-Portal/pullrequest/8654322
This PR just contains the localizable strings needed for that PR

@skapur12 skapur12 changed the title Add strings needed for portal localization Minor fixes and descriptive comments Sep 6, 2023
@skapur12 skapur12 merged commit 90fd62c into main Sep 6, 2023
6 checks passed
@skapur12 skapur12 deleted the skapur/portal-translations branch September 6, 2023 20:52
gingi pushed a commit that referenced this pull request Nov 9, 2023
* Compile package translations for use in the portal

* Remove unneeded strings
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.

2 participants