Skip to content

Fixup user guide#17926

Merged
seanbudd merged 4 commits into
nvaccess:betafrom
wmhn1872265132:FixupUserGuide
Apr 10, 2025
Merged

Fixup user guide#17926
seanbudd merged 4 commits into
nvaccess:betafrom
wmhn1872265132:FixupUserGuide

Conversation

@wmhn1872265132
Copy link
Copy Markdown
Contributor

@wmhn1872265132 wmhn1872265132 commented Apr 10, 2025

Link to issue number:

None

Summary of the issue:

While translating the user guide, I found some errors:

  1. PR Added short version for most commonly used command line options #17486 The added command line parameters are not recorded in the user guide
  2. The list rendering is incorrect
  3. There is an error in the title "Changing the automatic update channel"
  4. The title of the "{#AutomaticAddonUpdates}" section needs to be updated

Description of user facing changes

Fix the above issues

Description of development approach

Update User Guide

Testing strategy:

Read the document after building

Known issues with pull request:

None

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 April 10, 2025 01:47
@wmhn1872265132
Copy link
Copy Markdown
Contributor Author

By the way, The translation of nvda.po has never been uploaded to Crowdin. I don’t know if it was intentional or something went wrong?

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.

Thanks @wmhn1872265132!

@seanbudd seanbudd merged commit baa05ae into nvaccess:beta Apr 10, 2025
2 checks passed
@github-actions github-actions Bot added this to the 2025.2 milestone Apr 10, 2025
@wmhn1872265132 wmhn1872265132 deleted the FixupUserGuide branch April 10, 2025 05:48
@seanbudd seanbudd modified the milestones: 2025.2, 2025.1 Apr 10, 2025
@seanbudd
Copy link
Copy Markdown
Member

By the way, The translation of nvda.po has never been uploaded to Crowdin. I don’t know if it was intentional or something went wrong?

@wmhn1872265132 - this should be fixed now after #17955 builds in beta

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