docs: fix TypeScript import for Prompt types - #4842
Conversation
- Gate the 'Use native build server' UI option on isManagedCloud - Gate the enqueueBuild API path on isCloud() condition - Return clear error from API when feature not available - Add .server-changes/ file for server-only user-facing change Resolves triggerdotdev#4835
- Change import from direct type import to namespace import - PromptHandle, PromptIdentifier, PromptVariables are in the prompts namespace, not directly exported from @trigger.dev/sdk - Matches the pattern used throughout the rest of the documentation Resolves triggerdotdev#4812 (the TypeScript compilation issue)
|
|
Hi @MauryaQbit, thanks for your interest in contributing! This project requires that pull request authors are vouched, and you are not in the list of vouched users. This PR will be closed automatically. See https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md for more details. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
WalkthroughThe web application now detects managed cloud availability and disables the native build server option on self-hosted instances. The deployment service rejects native Git-based builds on non-cloud platforms with a ✨ Finishing Touches🧪 Generate unit tests (beta)
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 Warning |
1 participant
Resolves #4812 and #4824\n\n- Fixes TS2305 errors by using namespace import pattern\n- Changes import type { PromptHandle, PromptIdentifier, PromptVariables } to import type { prompts }\n- Updates type references to use prompts.PromptIdentifier and prompts.PromptVariables\n- Docs-only change, one issue, matches one-issue-per-PR rule\n\nVouch required: This is the contributor's first PR. A maintainer will need to vouch before merging.