Skip to content

Improve crowdinUpload jobs for GH Actions workflows#18333

Merged
seanbudd merged 1 commit intonvaccess:masterfrom
wmhn1872265132:FixupCrowdinUpload
Jun 26, 2025
Merged

Improve crowdinUpload jobs for GH Actions workflows#18333
seanbudd merged 1 commit intonvaccess:masterfrom
wmhn1872265132:FixupCrowdinUpload

Conversation

@wmhn1872265132
Copy link
Copy Markdown
Contributor

@wmhn1872265132 wmhn1872265132 commented Jun 25, 2025

Link to issue number:

follow up #18035

Summary of the issue:

The cache does not contain nvda.pot, additional steps are needed to download the file

Description of user facing changes:

Fix the above issue

Description of developer facing changes:

Fix the above issue

Description of development approach:

Add steps to download nvda.pot

Testing strategy:

Unfortunately we need to test this on production when we merge master to beta

Known issues with pull request:

Not know

Code Review Checklist:

  • Documentation:
    • Change log entry
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • API is compatible with existing add-ons.
  • Security precautions taken.

@coderabbitai summary

@wmhn1872265132 wmhn1872265132 requested a review from a team as a code owner June 25, 2025 09:31
@wmhn1872265132 wmhn1872265132 requested a review from seanbudd June 25, 2025 09:31
@wmhn1872265132 wmhn1872265132 marked this pull request as draft June 25, 2025 09:38
@wmhn1872265132 wmhn1872265132 marked this pull request as ready for review June 25, 2025 09:51
@seanbudd
Copy link
Copy Markdown
Member

@wmhn1872265132 - you linked a pull requests rather than an issue in your "fixes" comment. Could you rather say this is a "follow up" to that.

There's changes to this code happening in #18330 - can you remove the stuff around env/var changes? Otherwise this looks good to go, thanks for catching this

@seanbudd seanbudd added this to the 2025.2 milestone Jun 26, 2025
@wmhn1872265132
Copy link
Copy Markdown
Contributor Author

@seanbudd Done

Copy link
Copy Markdown
Member

@seanbudd seanbudd left a comment

Choose a reason for hiding this comment

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

@seanbudd seanbudd merged commit dfac937 into nvaccess:master Jun 26, 2025
11 of 12 checks passed
@wmhn1872265132 wmhn1872265132 deleted the FixupCrowdinUpload branch June 26, 2025 03:12
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