Skip to content

docs:update xpert translation docs#9

Merged
nsprenkle merged 5 commits intomasterfrom
nsprenkle/update-xpert-docs
Mar 12, 2026
Merged

docs:update xpert translation docs#9
nsprenkle merged 5 commits intomasterfrom
nsprenkle/update-xpert-docs

Conversation

@nsprenkle
Copy link
Copy Markdown
Member

@nsprenkle nsprenkle commented Mar 4, 2026

Updates the Xpert Course Translations feature description page (feature_description.rst) with revised copy and new screenshots reflecting the current learner-facing UI.

Changes:

  • Rewrote section 12.9.1 (Feature Description) with updated copy describing the XT feature from a learner perspective
  • Added new screenshots: globe icon in the top bar, language selection modal, and translation reset icon
  • Removed 6 outdated images that are no longer referenced

Date Needed (optional)

ASAP

Reviewers

Possible roles follow. The PR submitter checks the boxes after each reviewer finishes and gives 👍.

Testing

  • Ran ./run_tests.sh without warnings or errors

Copilot AI review requested due to automatic review settings March 4, 2026 18:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Xpert Course Translations (XT) documentation in the Course Authors guide to reflect the current learner-facing UI, including new screenshots, and adds a CLAUDE.md file with repository workflow/build guidance.

Changes:

  • Rewrites the XT Feature Description content to describe the learner experience (language selection + reset).
  • Adds/updates screenshot assets referenced by the updated doc.
  • Adds CLAUDE.md with common commands and repo/build notes for Claude Code.

Reviewed changes

Copilot reviewed 2 out of 11 changed files in this pull request and generated 2 comments.

File Description
en_us/course_authors/source/course_features/xpert_translations/feature_description.rst Updates XT feature description text and swaps screenshots to match current UI.
en_us/course_authors/source/images/xpert_translations_language_modal.png Adds/updates the language-selection modal screenshot used in the XT docs.
CLAUDE.md Documents repo build/test commands and high-level architecture notes for Claude Code usage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@nsprenkle nsprenkle force-pushed the nsprenkle/update-xpert-docs branch from d6cebc8 to 43e3d14 Compare March 4, 2026 18:56
Copilot AI review requested due to automatic review settings March 4, 2026 19:54
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 10 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copilot AI review requested due to automatic review settings March 4, 2026 20:06
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 12 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Why do we fail on warnings? That's stupid.
We should change this but, minimally, I will document it so it doesn't catch others off-guard in the process.
@nsprenkle nsprenkle force-pushed the nsprenkle/update-xpert-docs branch from 70e8733 to 9a950ce Compare March 4, 2026 20:17
Copilot AI review requested due to automatic review settings March 6, 2026 21:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 9 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@nsprenkle nsprenkle changed the title Nsprenkle/update xpert docs docs:update xpert translation docs Mar 9, 2026
Copy link
Copy Markdown

@jristau1984 jristau1984 left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@nsprenkle nsprenkle merged commit 8464092 into master Mar 12, 2026
5 checks passed
@nsprenkle nsprenkle deleted the nsprenkle/update-xpert-docs branch March 12, 2026 18:32
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