upd onboarding README #269

Merged
cirex-web merged 3 commits from staging into main 2026-09-12 19:38:06 +00:00
cirex-web commented 2026-09-11 19:02:26 +00:00 (Migrated from github.com)
No description provided.
coderabbitai[bot] commented 2026-09-11 19:02:36 +00:00 (Migrated from github.com)

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: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8ea066ab-31b4-4de4-b9fe-4d38ef6d404f

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.

<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: skip review by coderabbit.ai --> > [!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. > > <details> > <summary>⚙️ Run configuration</summary> > > **Configuration used**: Organization UI > > **Review profile**: CHILL > > **Plan**: Advanced > > **Run ID**: `8ea066ab-31b4-4de4-b9fe-4d38ef6d404f` > > </details> > > 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: > - [ ] <!-- {"checkboxId":"e9bb8d72-00e8-4f67-9cb2-caf3b22574fe"} --> 🔍 Trigger review <!-- end of auto-generated comment: skip review by coderabbit.ai --> <!-- tips_start --> --- Thanks for using [CodeRabbit](https://coderabbit.ai?utm_source=oss&utm_medium=github&utm_campaign=ScottyLabs/dining-api&utm_content=269)! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. <details> <summary>❤️ Share</summary> - [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai) - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code) </details> <sub>Comment `@coderabbitai help` to get the list of available commands.</sub> <!-- tips_end -->
copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2026-09-11 19:04:07 +00:00
copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

The README links should use HTTPS before approval.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the README with CMU Eats API links and onboarding resources.

Changes:

  • Adds production and staging API links.
  • Adds Notion and Slack onboarding information.
  • Retains setup instructions.
File summaries
File Summary Review notes
README.md Adds onboarding details and API references. Two nit comments (1 and 2 votes) request replacing HTTP links with HTTPS.
Review details

Suppressed comments (1)

README.md:4

  • These onboarding links use plain HTTP, so users can be exposed to an unencrypted connection or a tampered redirect before reaching the API. The API's own endpoint documentation already advertises https://api.cmueats.com; use HTTPS for both links here.
Visit the api [here](http://api.cmueats.com/).
Visit the staging api [here](http://api.staging.cmueats.com/).
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

### 🟡 Changes recommended The README links should use HTTPS before approval. *Once you've addressed the issues Copilot identified, you can request another Copilot review.* <details> <summary>Pull request overview</summary> Updates the README with CMU Eats API links and onboarding resources. **Changes:** - Adds production and staging API links. - Adds Notion and Slack onboarding information. - Retains setup instructions. </details> <details> <summary>File summaries</summary> | File | Summary | Review notes | |---|---|---| | `README.md` | Adds onboarding details and API references. | Two nit comments (1 and 2 votes) request replacing HTTP links with HTTPS. | </details> <details> <summary>Review details</summary> ### Suppressed comments (1) **README.md:4** * These onboarding links use plain HTTP, so users can be exposed to an unencrypted connection or a tampered redirect before reaching the API. The API's own endpoint documentation already advertises `https://api.cmueats.com`; use HTTPS for both links here. ``` Visit the api [here](http://api.cmueats.com/). Visit the staging api [here](http://api.staging.cmueats.com/). ``` - **Files reviewed:** 1/1 changed files - **Comments generated:** 1 - **Review effort level:** Lite </details> --- 💡 <a href="/ScottyLabs/dining-api/new/main?filename=.github/skills/code-review/SKILL.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add a `code-review` agent skill</a> or configure MCP servers for context-aware, tailored reviews. <a href="https://docs.github.com/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review?tool=webui#mcp-servers-and-agent-skills" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn more in the docs.</a>
@ -1,6 +1,11 @@
# Dining API
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2026-09-11 19:04:07 +00:00

This onboarding link uses unencrypted HTTP even though the repository's canonical API reference uses https://api.cmueats.com/v2/locations (src/endpoints/deprecated.ts:8). Linking contributors to HTTP can expose or downgrade their API traffic; please publish the production link over HTTPS.

This onboarding link uses unencrypted HTTP even though the repository's canonical API reference uses `https://api.cmueats.com/v2/locations` (`src/endpoints/deprecated.ts:8`). Linking contributors to HTTP can expose or downgrade their API traffic; please publish the production link over HTTPS.
cirex-web (Migrated from github.com) reviewed 2026-09-12 19:36:33 +00:00
@ -1,6 +1,11 @@
# Dining API
cirex-web (Migrated from github.com) commented 2026-09-12 19:36:27 +00:00

fixed

fixed
Arom1a (Migrated from github.com) approved these changes 2026-09-12 19:37:48 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
ScottyLabs/dining-api!269
No description provided.