Skip to content

Issue 53055: Check for multiple values in single value column#1794

Merged
labkey-nicka merged 6 commits intodevelopfrom
fb_paste_lookup_53055
May 23, 2025
Merged

Issue 53055: Check for multiple values in single value column#1794
labkey-nicka merged 6 commits intodevelopfrom
fb_paste_lookup_53055

Conversation

@labkey-nicka
Copy link
Copy Markdown
Contributor

@labkey-nicka labkey-nicka commented May 20, 2025

Rationale

This addresses Issue 53055 which updates the processing of values pasted into an editable grid to determine if the underlying column accepts multiple values before processing pasted values.

Related Pull Requests

Changes

  • Check if multiple values and the column supports multiple values when processing parsed values
  • Update tests to new expectations and add multi-value lookup column coverage

Copy link
Copy Markdown
Contributor

@labkey-alan labkey-alan left a comment

Choose a reason for hiding this comment

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

Looks good, as discussed this change is also needed in order to allow us to handle multi-line copy/paste, but we'll tackle that later.

@labkey-nicka labkey-nicka force-pushed the fb_paste_lookup_53055 branch from c33ec73 to 595c13c Compare May 23, 2025 19:46
@labkey-nicka labkey-nicka merged commit f04f3ef into develop May 23, 2025
3 checks passed
@labkey-nicka labkey-nicka deleted the fb_paste_lookup_53055 branch May 23, 2025 20:05
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.

2 participants