Skip to content

Update access-analyzer-upgrade-faq.md#890

Merged
hilram7 merged 3 commits into
devfrom
naa-kb-upgrade-faq-typo
May 12, 2026
Merged

Update access-analyzer-upgrade-faq.md#890
hilram7 merged 3 commits into
devfrom
naa-kb-upgrade-faq-typo

Conversation

@tay-caliguiri
Copy link
Copy Markdown
Contributor

fixed typo

@tay-caliguiri tay-caliguiri requested a review from a team as a code owner May 11, 2026 17:40
hilram7 added 2 commits May 12, 2026 14:53
…rose

- Fix products field: access-analyzer → accessanalyzer
- Add required kb tag to tags field
- Rename title and H1 to "Upgrade and Migration FAQ" (remove product name per style guide)
- Fix "Operation System" → "Operating System"
- Remove first-person plural ("we") across answers 1–2 (Vale)
- Remove "please" from answer 3 (Vale)
- Fix passive voice and idiom "spinning up" in answer 1 (Dale)
Copy link
Copy Markdown
Collaborator

@hilram7 hilram7 left a comment

Choose a reason for hiding this comment

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

KB Review

Files reviewed: 1

Changes applied:

  • Fixed products field: access-analyzer → accessanalyzer (invalid product ID)
  • Added required kb tag to tags field
  • Renamed title, H1, and sidebar_label from "Access Analyzer Upgrade FAQ" to "Upgrade and Migration FAQ" (product name removed per style guide; title updated to reflect
    migration content)
  • Fixed "Operation System" → "Operating System"
  • Rewrote answers 1 and 2 to remove first-person plural ("we") (Vale)
  • Removed "please" from answer 3 (Vale)
  • Fixed passive voice and idiomatic phrasing ("spinning up") in answer 1 (Dale)
  • Added pathname:// links for both "SQL Server Requirements" references in answer 3 and the Related Articles section

Local testing:

  • Vale: 0 errors, 0 warnings after fixes
  • Dev server: link to SQL Server Requirements verified working

@hilram7 hilram7 merged commit 3130926 into dev May 12, 2026
6 checks passed
@hilram7 hilram7 deleted the naa-kb-upgrade-faq-typo branch May 12, 2026 19:16
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