Cloning within course #457
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThe PR adds a new subsection to the grade-book documentation that explains cloning a rubric between assignments within the same course. The addition includes step-by-step instructions, supporting images, and a minor text capitalization update from "yes" to "Yes." Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Fix all issues with AI agents
In `@source/instructors/authoring/assessments/grade-book.rst`:
- Line 118: Replace the lowercase single-word term "gradebook" with the
two-word, capitalized term "Grade Book" wherever it appears in the sentence
"Navigate to your gradebook assessment using Method 2 from the \"Grading a Grade
Book Assessment\" section" (and the similar occurrence noted nearby) so the
document consistently uses "Grade Book" throughout.
- Line 118: Replace the quoted section reference "Grading a Grade Book
Assessment" on the line that says "Navigate to your gradebook assessment using
Method 2 from the \"Grading a Grade Book Assessment\" section" with an RST
cross-reference using :ref:, e.g. :ref:`grading-grade-book`, and add a target
label (e.g. .. _grading-grade-book:) at the start of the "Grading a Grade Book
Assessment" section if it doesn’t exist; mirror the existing usage pattern found
on line 43 to ensure the link becomes a proper hyperlink and remains
maintainable.
- Line 133: Replace the capitalized confirmation label "Yes" with the lowercase
"yes" to match the document's existing usage; locate the sentence "Confirm your
changes by clicking **Yes**" in the grade-book.rst content and change the markup
to "Confirm your changes by clicking **yes**" so it is consistent with the other
occurrences at lines referencing the same confirmation action.
|
|
||
| To clone a rubric from one assignment to another within the same course, you must first add at least one rubric item to the source rubric. Follow these steps: | ||
|
|
||
| 1. Navigate to your gradebook assessment using Method 2 from the "Grading a Grade Book Assessment" section |
There was a problem hiding this comment.
Use consistent terminology: "Grade Book" instead of "gradebook".
Lines 118 and 120 use "gradebook" (one word, lowercase), but the entire document consistently uses "Grade Book" (two words, capitalized). Maintain consistency with the established terminology.
📝 Proposed fix for terminology consistency
-1. Navigate to your gradebook assessment using Method 2 from the "Grading a Grade Book Assessment" section
+1. Navigate to your Grade Book assessment using Method 2 from the "Grading a Grade Book Assessment" section-3. Navigate to a different gradebook assessment in the same course using the same method
+3. Navigate to a different Grade Book assessment in the same course using the same methodAlso applies to: 120-120
🤖 Prompt for AI Agents
In `@source/instructors/authoring/assessments/grade-book.rst` at line 118, Replace
the lowercase single-word term "gradebook" with the two-word, capitalized term
"Grade Book" wherever it appears in the sentence "Navigate to your gradebook
assessment using Method 2 from the \"Grading a Grade Book Assessment\" section"
(and the similar occurrence noted nearby) so the document consistently uses
"Grade Book" throughout.
🧹 Nitpick | 🔵 Trivial
Consider using RST cross-reference instead of quoted section name.
Line 118 references another section using quoted text. Consider using the :ref: directive (similar to line 43) to create a proper hyperlink and improve maintainability. If the section doesn't have a label, adding one would enable better cross-referencing.
📝 Suggested improvement with cross-reference
If a label is added to the "Grading a Grade Book Assessment" section (e.g., .. _grading-grade-book:), the reference could be:
-1. Navigate to your Grade Book assessment using Method 2 from the "Grading a Grade Book Assessment" section
+1. Navigate to your Grade Book assessment using Method 2 from :ref:`grading-grade-book`Alternatively, keep the current approach if cross-referencing is not a priority for this documentation.
🤖 Prompt for AI Agents
In `@source/instructors/authoring/assessments/grade-book.rst` at line 118, Replace
the quoted section reference "Grading a Grade Book Assessment" on the line that
says "Navigate to your gradebook assessment using Method 2 from the \"Grading a
Grade Book Assessment\" section" with an RST cross-reference using :ref:, e.g.
:ref:`grading-grade-book`, and add a target label (e.g. .. _grading-grade-book:)
at the start of the "Grading a Grade Book Assessment" section if it doesn’t
exist; mirror the existing usage pattern found on line 43 to ensure the link
becomes a proper hyperlink and remains maintainable.
| :alt: Rubric list | ||
| :width: 450px | ||
|
|
||
| 6. Confirm your changes by clicking **Yes** No newline at end of file |
There was a problem hiding this comment.
Maintain consistent capitalization: "yes" instead of "Yes".
Line 133 uses "Yes" (capitalized), but lines 92 and 110 consistently use "yes" (lowercase) for the same confirmation action. Standardize the capitalization throughout the document.
📝 Proposed fix for capitalization consistency
-6. Confirm your changes by clicking **Yes**
+6. Confirm your changes by clicking **yes**📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| 6. Confirm your changes by clicking **Yes** | |
| 6. Confirm your changes by clicking **yes** |
🤖 Prompt for AI Agents
In `@source/instructors/authoring/assessments/grade-book.rst` at line 133, Replace
the capitalized confirmation label "Yes" with the lowercase "yes" to match the
document's existing usage; locate the sentence "Confirm your changes by clicking
**Yes**" in the grade-book.rst content and change the markup to "Confirm your
changes by clicking **yes**" so it is consistent with the other occurrences at
lines referencing the same confirmation action.
1 participant
Summary by CodeRabbit