Skip to content

Mentor role clarifications Clarify instructor roles and password reset documentation review#454

Merged
shajason merged 2 commits intomasterfrom
mentorrole
Jan 29, 2026
Merged

Mentor role clarifications Clarify instructor roles and password reset documentation review#454
shajason merged 2 commits intomasterfrom
mentorrole

Conversation

@shajason
Copy link
Collaborator

@shajason shajason commented Jan 28, 2026

Summary by CodeRabbit

  • Documentation
    • Clarified password reset assistance procedures for course instructors.
    • Corrected punctuation in the role table.
    • Added detail that teachers enrolled as Mentors have read-only access in the organization.
    • Expanded guidance on restricting Mentor access to specific students in the course.

✏️ Tip: You can customize this high-level summary in your review settings.

@shajason shajason requested a review from LolaValente January 28, 2026 22:12
@coderabbitai
Copy link

coderabbitai bot commented Jan 28, 2026

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai bot changed the title Mentor role clarifications @coderabbitai review Mentor role clarifications Clarify instructor roles and password reset documentation review Jan 28, 2026
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@source/instructors/admin/integration/lms-students.rst`:
- Around line 60-62: The two list items "Teachers who are enrolled with the
Mentor role are designated as read-only teachers in the organization." and "If
you want Mentors to have access to specific students only, rather than all
students in the course, see :ref:`Filter Learners For Mentors
<filter-learners>`." are separated by an unnecessary blank line; remove the
empty line between these bullets so the list items are contiguous and the list
formatting is compact and consistent.
- Around line 40-41: The note's second sentence is a comma splice and unclear;
edit the note block in source/instructors/admin/integration/lms-students.rst so
the two sentences read cleanly and the reference remains correct — for example:
"If you need to reset your password, you can use your username instead of your
email address; course instructors can assist — see :ref:`reset-pass`." Ensure
you remove the stray comma after "instructors", keep the :ref:`reset-pass`
target, and preserve the note directive formatting.

@shajason shajason merged commit d31cf7b into master Jan 29, 2026
1 check passed
@shajason shajason deleted the mentorrole branch January 29, 2026 14:14
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