Conversation
WhyRoy
force-pushed
the
feat/prevent-duplicate-attachment-paste
branch
from
September 15, 2026 15:21
003ba41 to
396ed56
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Background
When pasting an image into an attachment cell, the same browser paste event can be processed by both the attachment upload zone and the grid editor. This may result in the same image being uploaded twice.
This change makes the attachment upload zone consume file paste events after handling them, so the grid editor does not process the same file paste again. Regular text paste behavior and repeated independent pastes remain available.
Description
When a user pastes an image directly into an attachment cell, the same
clipboard event may be processed twice:
As a result, one pasted image can appear as two uploaded attachments.
This change makes the attachment upload zone consume file paste events after
successfully handling file uploads. The event no longer reaches the grid paste
handler, preventing duplicate uploads.
Changes
Reproduction
Ctrl+V.Expected Behavior
Each file in a single clipboard paste is uploaded exactly once.
Testing
development environment.