Skip to content

Add link to referenced docs in document view#70

Merged
vkarpov15 merged 2 commits intomainfrom
codex/add-link-to-referenced-document-in-document-details.html
Jul 14, 2025
Merged

Add link to referenced docs in document view#70
vkarpov15 merged 2 commits intomainfrom
codex/add-link-to-referenced-document-in-document-details.html

Conversation

@vkarpov15
Copy link
Copy Markdown
Member

Summary

  • enable navigation to referenced documents from the document details page using <router-link>

Testing

  • npm run lint
  • npm test (fails: Timeout of 2000ms exceeded)

https://chatgpt.com/codex/tasks/task_e_687564777220832488393744cc41af2d

@vkarpov15 vkarpov15 marked this pull request as ready for review July 14, 2025 21:17
@vkarpov15 vkarpov15 requested a review from Copilot July 14, 2025 21:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a direct link to navigate to referenced documents from the document details view, using <router-link>.

  • Wraps the path display in a flex container with space-between alignment.
  • Introduces a conditional <router-link> that points to /model/{ref}/document/{id} when path.ref and the value for the path exist.
  • Adjusts layout to accommodate the new link without affecting existing date-edit UI.
Comments suppressed due to low confidence (1)

frontend/src/document-details/document-property/document-property.html:11

  • There are no tests covering this new link behavior. Add unit tests to verify that the router-link renders when path.ref and the value exist, and that it navigates to the correct path.
        <router-link

@vkarpov15 vkarpov15 merged commit 1a07597 into main Jul 14, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants