move to app.credal.ai - #376
jackfischer wants to merge 1 commit into
Conversation
Greptile SummaryThis PR migrates all API base URLs from Confidence Score: 5/5Safe to merge — all domain references updated consistently with no regressions introduced. Only P2 findings remain, both flagging pre-existing URL path mismatches that were not introduced by this PR. The domain migration itself is complete and correct. No files require special attention; the two P2 comments in audit-logs-api.mdx are pre-existing issues worth a follow-up but not blockers. Important Files Changed
Prompt To Fix All With AIThis is a comment left during a code review.
Path: fern/docs/pages/administration/audit-logs-api.mdx
Line: 30
Comment:
**Base URL and cURL example path mismatch (pre-existing)**
The Base URL on line 15 includes `/v0/` (`/api/v0/audit-logs/organization`), but the cURL example below omits it (`/api/audit-logs/organization`). Since you're already touching these URLs, it's worth aligning them so readers use the correct path.
```suggestion
curl 'https://app.credal.ai/api/v0/audit-logs/organization' \
```
How can I resolve this? If you propose a fix, please make it concise.
---
This is a comment left during a code review.
Path: fern/docs/pages/administration/audit-logs-api.mdx
Line: 111
Comment:
**Base URL and cURL example path segment order differ (pre-existing)**
The Base URL on line 96 is `/api/audit-logs/v0/usage-analytics/`, but the cURL example has the segments in a different order: `/api/v0/audit-logs/usage-analytics`. One of these is likely wrong; since you're already touching these URLs, it's a good moment to verify and align them.
How can I resolve this? If you propose a fix, please make it concise.Reviews (1): Last reviewed commit: "done" | Re-trigger Greptile |
| **Example cURL request** | ||
| ```bash | ||
| curl 'https://api.credal.ai/api/audit-logs/organization' \ | ||
| curl 'https://app.credal.ai/api/audit-logs/organization' \ |
There was a problem hiding this comment.
Base URL and cURL example path mismatch (pre-existing)
The Base URL on line 15 includes /v0/ (/api/v0/audit-logs/organization), but the cURL example below omits it (/api/audit-logs/organization). Since you're already touching these URLs, it's worth aligning them so readers use the correct path.
| curl 'https://app.credal.ai/api/audit-logs/organization' \ | |
| curl 'https://app.credal.ai/api/v0/audit-logs/organization' \ |
Prompt To Fix With AI
This is a comment left during a code review.
Path: fern/docs/pages/administration/audit-logs-api.mdx
Line: 30
Comment:
**Base URL and cURL example path mismatch (pre-existing)**
The Base URL on line 15 includes `/v0/` (`/api/v0/audit-logs/organization`), but the cURL example below omits it (`/api/audit-logs/organization`). Since you're already touching these URLs, it's worth aligning them so readers use the correct path.
```suggestion
curl 'https://app.credal.ai/api/v0/audit-logs/organization' \
```
How can I resolve this? If you propose a fix, please make it concise.| **Example cURL request** | ||
| ```bash | ||
| curl 'https://api.credal.ai/api/v0/audit-logs/usage-analytics' \ | ||
| curl 'https://app.credal.ai/api/v0/audit-logs/usage-analytics' \ |
There was a problem hiding this comment.
Base URL and cURL example path segment order differ (pre-existing)
The Base URL on line 96 is /api/audit-logs/v0/usage-analytics/, but the cURL example has the segments in a different order: /api/v0/audit-logs/usage-analytics. One of these is likely wrong; since you're already touching these URLs, it's a good moment to verify and align them.
Prompt To Fix With AI
This is a comment left during a code review.
Path: fern/docs/pages/administration/audit-logs-api.mdx
Line: 111
Comment:
**Base URL and cURL example path segment order differ (pre-existing)**
The Base URL on line 96 is `/api/audit-logs/v0/usage-analytics/`, but the cURL example has the segments in a different order: `/api/v0/audit-logs/usage-analytics`. One of these is likely wrong; since you're already touching these URLs, it's a good moment to verify and align them.
How can I resolve this? If you propose a fix, please make it concise.
No description provided.