Skip to content

Expose more PropertyDescriptor data in exp.Fields#7130

Merged
labkey-jeckels merged 7 commits intodevelopfrom
fb_moreFields
Oct 22, 2025
Merged

Expose more PropertyDescriptor data in exp.Fields#7130
labkey-jeckels merged 7 commits intodevelopfrom
fb_moreFields

Conversation

@labkey-jeckels
Copy link
Contributor

@labkey-jeckels labkey-jeckels commented Oct 16, 2025

Rationale

In order to find fields of type Flag, we need to expose the ConceptURI.

Changes

  • Add ConceptURI to exp.Fields and exp.PhiFields
  • Expose all of the other columns as hidden columns

Tasks 📍

  • Manual Testing @labkey-adam
    • Tested both Fields and PhiFields, verified ConceptURI looked correct, added all hidden columns, applied container filters (e.g., All Folders), filtered/sorted on various columns
  • Needs Automation - Not Needed

@labkey-jeckels labkey-jeckels self-assigned this Oct 16, 2025
@labkey-jeckels
Copy link
Contributor Author

Are any of these columns sensitive enough to avoid exposing, or only to certain users? I didn't see anything concerning when I scanned the list.

Copy link
Contributor

@labkey-adam labkey-adam left a comment

Choose a reason for hiding this comment

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

Need to fix PhiFieldsTable to not blow up, but no need for follow-on review

@labkey-jeckels labkey-jeckels merged commit 0b8d3cc into develop Oct 22, 2025
6 of 14 checks passed
@labkey-jeckels labkey-jeckels deleted the fb_moreFields branch October 22, 2025 18:14
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