Skip to content

just sandboxes and course copy - #475

Merged
shajason merged 2 commits into
masterfrom
llmfreetextnew
Aug 7, 2026
Merged

shajason merged 2 commits into
masterfrom
llmfreetextnew

Conversation

@shajason

@shajason shajason commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Jairo - could you please just look at the changes I made to the sandboxes page. I will finish everything else next week but I want to get this in.

Summary by CodeRabbit

  • Documentation
    • Simplified LTI course copy setup by removing the prerequisite to enable Constant URLs.
    • Documented optional sandbox warmup storage, including EBS preparation and variable readiness times.
    • Clarified course search and sorting behavior, including tag searches and Owner-only visibility for the All tab.

@shajason
shajason requested a review from jairovelasquez August 7, 2026 12:51
@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

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.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a4711615-6ff5-4aa3-b112-862211e5c972

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

Use the checkbox below for a quick retry:

  • 🔍 Trigger 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.

Comment thread source/instructors/admin/sandboxes.rst Outdated
Comment thread source/instructors/admin/sandboxes.rst Outdated
jairovelasquez
jairovelasquez previously approved these changes Aug 7, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@source/instructors/admin/sandboxes.rst`:
- Around line 242-245: Update the Type cells for warmup_storage in both the
admin sandbox and Jupyter tables from enum to boolean, preserving the documented
true/false values as unquoted YAML booleans.
- Around line 57-58: Update the Warmup Storage description to replace the comma
splice and explicitly state that warmup storage becomes ready after the
configuration is published, with the Windows and GPU Jupyter durations presented
as readiness estimates.

In `@source/instructors/setupcourses/search-course.rst`:
- Around line 33-35: Update the course-search guidance near the name and tag
examples to use direct, concise instructions rather than describing what Codio
does. Clearly instruct users how to search by course name and how to use the
tag:"Intro" query, preserving the existing search syntax.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 1667a1c6-a0f0-4d69-9fa1-4bb2a2af0e89

📥 Commits

Reviewing files that changed from the base of the PR and between b7090e4 and c3212b2.

⛔ Files ignored due to path filters (1)
  • source/img/lti/enable_class_fork.png is excluded by !**/*.png, !**/*.png
📒 Files selected for processing (3)
  • source/instructors/admin/integration/lti-course-copy.rst
  • source/instructors/admin/sandboxes.rst
  • source/instructors/setupcourses/search-course.rst
💤 Files with no reviewable changes (1)
  • source/instructors/admin/integration/lti-course-copy.rst

Comment thread source/instructors/admin/sandboxes.rst Outdated
Comment thread source/instructors/admin/sandboxes.rst
Comment thread source/instructors/setupcourses/search-course.rst Outdated
@shajason
shajason removed the request for review from LolaValente August 7, 2026 17:10
@shajason
shajason merged commit b2302ca into master Aug 7, 2026
1 check passed
@shajason
shajason deleted the llmfreetextnew branch August 7, 2026 17:11
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