Skip to content

Fixed the client id values for Blackboard#412

Merged
shajason merged 1 commit intomasterfrom
blackboardid
Oct 6, 2025
Merged

Fixed the client id values for Blackboard#412
shajason merged 1 commit intomasterfrom
blackboardid

Conversation

@shajason
Copy link
Collaborator

@shajason shajason commented Oct 6, 2025

Summary by CodeRabbit

  • Documentation
    • Updated the Blackboard LTI 1.3 integration guide with the latest client IDs for codio.com and codio.co.uk.
    • Refreshed example snippets to reflect current identifiers.
    • Clarified setup steps to reduce configuration errors during integration.
    • No functional or behavioral changes to the product; this is a documentation accuracy update only.

@coderabbitai
Copy link

coderabbitai bot commented Oct 6, 2025

Walkthrough

Updated two literal client ID strings in the Blackboard LTI 1.3 integration documentation for codio.com and codio.co.uk. No procedural or structural documentation changes.

Changes

Cohort / File(s) Summary
Docs — Blackboard LTI 1.3 client IDs
`source/instructors/admin/integration/lti1-3Blackboard.rst`
Replaced codio.com client ID with 0128e3c6-67a1-4bfb-9881-747baf0c7105 and codio.co.uk client ID with 20d13eaa-8d29-4763-87ab-8137aa94129f in literal code blocks.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • LTI and DynReg Update #406 — Also updates the same Blackboard LTI 1.3 integration doc, likely addressing related identifier or configuration text changes.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title accurately and concisely describes the main change of updating the client ID values for Blackboard and directly corresponds to the modifications in the documentation.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch blackboardid

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7757de4 and dafa8b7.

📒 Files selected for processing (1)
  • source/instructors/admin/integration/lti1-3Blackboard.rst (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.rst

⚙️ CodeRabbit configuration file

**/*.rst: Review files for:

  • Consistent formatting (e.g., headings, lists, links).
  • Clear and concise language.
  • Correct grammar and spelling.
  • Proper use of rst syntax (e.g., avoid broken links or invalid code blocks).
  • Adherence to style guides (e.g., consistent tone, terminology).
    Suggest improvements to enhance readability and accessibility.

Files:

  • source/instructors/admin/integration/lti1-3Blackboard.rst

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.

@shajason shajason merged commit f558f1a into master Oct 6, 2025
1 check passed
@shajason shajason deleted the blackboardid branch October 6, 2025 14:13
@coderabbitai coderabbitai bot mentioned this pull request Oct 17, 2025
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