Skip to content

Fix dead "startupShutdown" link in technical docs#14567

Merged
seanbudd merged 1 commit into
nvaccess:masterfrom
sivakusayan:patch-1
Jan 18, 2023
Merged

Fix dead "startupShutdown" link in technical docs#14567
seanbudd merged 1 commit into
nvaccess:masterfrom
sivakusayan:patch-1

Conversation

@sivakusayan

@sivakusayan sivakusayan commented Jan 18, 2023

Copy link
Copy Markdown
Contributor

Link to issue number:

N/A

Summary of the issue:

Previously, in the "Technical Design Overview" document, the startupShutdown link under the Launcher heading didn't work.

Description of user facing changes

Now, the startupShutdown link should correctly link to NVDA Starting and Exiting outline page.

Description of development approach

N/A

Testing strategy:

Used Github's "Preview" feature when editing technicalDesignOverview.md file to make sure the link opens up correctly.

Known issues with pull request:

N/A

Change log entries:

N/A - this is a small technical documentation fix.

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
  • Security precautions taken.

Found a dead link when reading the documentation, this should be fixed now.
@sivakusayan sivakusayan marked this pull request as ready for review January 18, 2023 16:35
@sivakusayan sivakusayan requested a review from a team as a code owner January 18, 2023 16:35
@sivakusayan sivakusayan requested a review from seanbudd January 18, 2023 16:35
@josephsl

josephsl commented Jan 18, 2023 via email

Copy link
Copy Markdown
Contributor

@sivakusayan

Copy link
Copy Markdown
Contributor Author

Apologies, I thought that outline would be noise for reviewers since this is a relatively small change to documentation, but it should be filled out now.

@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.

Thanks for this fix!

@seanbudd seanbudd merged commit 7cc0deb into nvaccess:master Jan 18, 2023
@nvaccessAuto nvaccessAuto added this to the 2023.1 milestone Jan 18, 2023
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.

4 participants