Skip to content

Update LSID metadata to avoid making it unselectable #363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
labkey-bpatel merged 1 commit into from
Apr 22, 2022
Merged

Update LSID metadata to avoid making it unselectable #363

labkey-bpatel merged 1 commit into from
Apr 22, 2022

Conversation

@labkey-bpatel
Copy link
Contributor

@labkey-bpatel labkey-bpatel commented Apr 22, 2022 •
edited
Loading

Rationale

Submitting a wnprc purchasing request form with extensible columns throws "java.lang.IllegalStateException: LSID value not found in table - purchasingRequests".

Its a regression from changes made w.r.t extensible columns and TableSelector ignoring unselectable columns:
LabKey/platform#3151
LabKey/platform#3227

Related Pull Requests

Changes

  • Update LSID metadata to set 'isUnselectable' to false.

@labkey-bpatel labkey-bpatel mentioned this pull request Apr 22, 2022
Merged
@labkey-bpatel labkey-bpatel merged commit 44688f2 into develop Apr 22, 2022
@labkey-bpatel labkey-bpatel deleted the fb_ehr_purchasing_update_lsid_metadata branch April 22, 2022 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@labkey-jeckels labkey-jeckels labkey-jeckels approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@labkey-bpatel @labkey-jeckels