Skip to content

[OGUI-1934] Create clipboard utility - #3694

Merged
isaachilly merged 3 commits into
devfrom
improv/FRM/OGUI-1934/Centralise-clipboard-behaviour
Oct 26, 2026
Merged

isaachilly merged 3 commits into
devfrom
improv/FRM/OGUI-1934/Centralise-clipboard-behaviour

Conversation

@isaachilly

@isaachilly isaachilly commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected
  • FLP integration tests were ran successful

Ticket:

Given the ticket OGUI-1906 wants to add a copy button in notifications, it is best to centralise clipboard logic so we do not repeat ourselves and/or miss certain safety guards.

Code:

Extracts clipboard logic into a utility that can simply be imported by what copy element or function needs it.

Reviewer:

Could locate a copyToClipboardComponent and test the functionality via that.

@isaachilly isaachilly self-assigned this Oct 21, 2026
@isaachilly
isaachilly added this pull request to stack #3696 October 22, 2026 13:03
Expose copyToClipboard and getClipboardUnavailableReason that can be used by clipboard wanting components.
CopyToClipboardComponent now uses copyToClipboard instead of its own availability checks.
@isaachilly
isaachilly force-pushed the improv/FRM/OGUI-1934/Centralise-clipboard-behaviour branch from 24323a2 to 3938049 Compare October 22, 2026 14:16
@isaachilly isaachilly changed the title [OGUI-1934] Centralise clipboard behaviour [OGUI-1934] Create clipboard utility Oct 22, 2026
@isaachilly
isaachilly marked this pull request as ready for review October 22, 2026 15:27
@isaachilly
isaachilly requested a review from graduta as a code owner October 22, 2026 15:27
@isaachilly
isaachilly merged commit e6b590d into dev Oct 26, 2026
14 checks passed
@isaachilly
isaachilly deleted the improv/FRM/OGUI-1934/Centralise-clipboard-behaviour branch October 26, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants