fix: update ParagraphDocumentContent rendering condition in index.vue - #6594
Conversation
|
Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information |
There was a problem hiding this comment.
🟢 Ready to approve
The change fixes an objectively incorrect v-else pairing that could render the wrong component for non-execution-detail panel types, and the import cleanup is consistent and safe.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
Fixes right-side panel rendering in the PC chat view by ensuring ParagraphDocumentContent is only rendered when the active panel type is paragraphDocument, preventing it from incorrectly showing up for other panel types.
Changes:
- Replace the
v-elsefallback with an explicitv-if="rightPanelType === 'paragraphDocument'"forParagraphDocumentContent. - Remove the unused
watchimport and tidy thevuenamed import ordering.
File summaries
| File | Description |
|---|---|
| ui/src/views/chat/pc/index.vue | Corrects right panel conditional rendering so paragraph document content only appears for the paragraph-document panel type. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
fix: update ParagraphDocumentContent rendering condition in index.vue