Skip to content

Cherry pick from RC#13185

Merged
feerrenrut merged 9 commits into
betafrom
cherryPickFromRC
Dec 22, 2021
Merged

Cherry pick from RC#13185
feerrenrut merged 9 commits into
betafrom
cherryPickFromRC

Conversation

@feerrenrut

Copy link
Copy Markdown
Contributor

Link to issue number:

None

Summary of the issue:

Initially changes for the point release were delivered to beta, subsequently it was decided to avoid complications with translations, to make changes directly on RC.
The changes were cherry picked from beta.
There were then several more changes made squashed / merged to RC
These changes need to be cherry picked back into the beta branch.

Description of how this pull request fixes the issue:

  • Cherry pick (and recreate one merge) changes on RC not on Beta
  • Fast forward beta to the head of this branch

Testing strategy:

Check that the diff between the release tag an this branch is only the translations (already on beta):

git diff release-2021.3.1 --stat
 source/locale/de/LC_MESSAGES/nvda.po |   20 +-
 source/locale/de/symbols.dic         |    6 +-
 source/locale/fr/symbols.dic         |    1 +
 source/locale/ka/symbols.dic         |  156 +-
 source/locale/sl/LC_MESSAGES/nvda.po | 9621 ++++++++++++++++++----------------
 source/locale/sl/symbols.dic         |  602 ++-
 source/locale/tr/LC_MESSAGES/nvda.po |   35 +-
 source/locale/vi/LC_MESSAGES/nvda.po |    8 +-
 source/locale/zh_CN/symbols.dic      |  200 +-
 user_docs/tr/userGuide.t2t           |   12 +-
 user_docs/vi/changes.t2t             |    2 +-
 user_docs/zh_CN/userGuide.t2t        |    2 +-
 user_docs/zh_TW/userGuide.t2t        |    2 +-
 13 files changed, 5745 insertions(+), 4922 deletions(-)

Known issues with pull request:

  • RC will need to be reset to beta prior to the next release.

Change log entries:

None

Code Review Checklist:

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

feerrenrut and others added 9 commits December 22, 2021 13:47
Add type information and fix port listing for seikantk driver

Move registration of seikantk to bdDetect
  - This reflects how the other drivers are registered

Fall back to base implementation for check method
All other drivers for which check is shadowed either do not support
auto-detection yet or cannot be detected passively
i.e. because they're using a serial connection.
Duplication of the string makes it a "magic value" and hard to trace its
meaning.
Explicitly map to BrailleDisplayDriver on the module.
Ensure there is no confusion with KEY_HID="hid" in bdDetect.py
* add setting to disable HID Braille
@feerrenrut feerrenrut requested a review from a team as a code owner December 22, 2021 06:20
@feerrenrut feerrenrut requested review from a team and seanbudd and removed request for a team December 22, 2021 06:20

@seanbudd seanbudd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I performed git diff origin/cherryPickFromRC release-2021.3.1 --stat and confirmed that the only changes are translations

@feerrenrut

Copy link
Copy Markdown
Contributor Author

When the build completes I'll do the fast forward merge.

@feerrenrut feerrenrut merged commit f642b6a into beta Dec 22, 2021
@feerrenrut feerrenrut deleted the cherryPickFromRC branch December 22, 2021 06:44
@nvaccessAuto nvaccessAuto added this to the 2022.1 milestone Dec 22, 2021
@feerrenrut feerrenrut removed this from the 2022.1 milestone Dec 22, 2021
feerrenrut added a commit that referenced this pull request Dec 22, 2021
Beta to master

Includes changes from the point release 2021.3.1.
See also #13185 for further details.
@seanbudd seanbudd mentioned this pull request Feb 24, 2022
5 tasks
seanbudd added a commit that referenced this pull request Feb 24, 2022
Initially changes for 2021.3.1 were delivered to beta, subsequently it was decided to avoid complications with translations, to make changes directly on RC.
The changes were cherry picked from beta.
There were then several more changes made squashed / merged to RC
These changes need to be cherry picked back into the beta branch.
This was done in #13185

Changes for 2021.3.3 were cherrypicked directly to rc.
But because rc and beta/master have diverged, these changes need to be cherrypicked back to master and beta.
An exchange of merging master and beta has already occurred since the last release.
As a result, these changes need to be cherry-picked directly to master, to get eventually merged to beta.

Description of how this pull request fixes the issue:
Update the changes file from release-2021.3.3
No other introduced changes from 2021.3.2 and 2021.3.3 need to be cherry-picked back to master.
This is because 2021.3.2 and 2021.3.3 was created from code existing on master.
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