Skip to content

Conversation

@aonnikov
Copy link
Member

No description provided.

@aonnikov aonnikov requested review from Copilot and lexiv0re August 12, 2025 11:25
Copy link
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 generic "Copy Link" action that can be used across all document types by implementing reusable link copying functionality at the view level and removing document-specific implementations.

  • Introduces generic CopyLink action in the view plugin with supporting utility functions
  • Removes document-specific copy link implementations in favor of the generic solution
  • Updates existing copy link actions to override the new generic action

Reviewed Changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
plugins/view/src/index.ts Adds CopyLink action reference
plugins/view-resources/src/utils.ts Implements canCopyLink and getLink utility functions
plugins/view-resources/src/index.ts Exports new utility functions as resources
plugins/view-assets/lang/*.json Adds "Copy link" translations in multiple languages
plugins/document/src/plugin.ts Removes document-specific CopyDocumentLink action
plugins/document-resources/src/*.ts Removes document-specific link generation functions
models/view/src/*.ts Defines generic CopyLink action and supporting resources
models//src/.ts Updates existing copy link actions to override the generic one
models/card/package.json Adds model-guest dependency for public link functionality

lexiv0re
lexiv0re previously approved these changes Aug 12, 2025
Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
@aonnikov aonnikov merged commit 7d0034c into develop Aug 12, 2025
15 checks passed
@aonnikov aonnikov deleted the eqms-1635-copy-link branch August 12, 2025 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants