CRDCDH-3602 Data Submissions List – Use study.dbGaPID instead of root dbGaPID

[amattu2]

Overview

This PR addresses a request from the BE team to use study.dbGaPID instead of submission.dbGaPID, since study.dbGaPID is used for the filters. Technically they should always be identical

Change Details (Specifics)

• Switch the Data Submission List view from reading the root dbGaPID object to the nested study dbGaPID
• Mark the root dbGaPID property as deprecated – It's still technically populated though

Related Ticket(s)

CRDCDH-3602 (Task)

[relativeci]

#420 Bundle Size — 12.29MiB (~+0.01%).

960d0e2(current) vs 7804c52 3.6.0#419(baseline)

Warning

Bundle contains 50 duplicate packages – View duplicate packages

Bundle metrics   2 changes 1 regression

	Current #420	Baseline #419
Initial JS	874.56KiB(~+0.01%)	874.56KiB
Initial CSS	0B	0B
Cache Invalidation	6.12%	6.87%
Chunks	58	58
Assets	101	101
Modules	8891	8891
Duplicate Modules	0	0
Duplicate Code	0%	0%
Packages	272	272
Duplicate Packages	49	49

Bundle size by type   1 change 1 regression

	Current #420	Baseline #419
JS	9.77MiB (~+0.01%)	9.77MiB
IMG	1.72MiB	1.72MiB
Fonts	805.07KiB	805.07KiB
CSS	18KiB	18KiB

Bundle analysis report Branch CRDCDH-3602 Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[Copilot]

Pull request overview

This PR migrates dbGaPID usage in the data submissions list from the submission top-level field to the nested study.dbGaPID, while marking the top-level dbGaPID as deprecated in the shared Submission type.

Changes:

• Extend Submission.study to include dbGaPID, and deprecate the top-level Submission.dbGaPID field.
• Update the listSubmissions GraphQL query to fetch study.dbGaPID instead of the top-level dbGaPID.
• Update the Data Submissions list table column to display/sort/export using study.dbGaPID.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/types/Submissions.d.ts	Adds dbGaPID to Submission.study and deprecates top-level dbGaPID.
src/graphql/listSubmissions.ts	Moves dbGaPID selection under study in the query.
src/content/dataSubmissions/DataSubmissionsListView.tsx	Switches the “dbGaP ID” column to use study.dbGaPID for rendering/sorting/export.

---

You can also share your feedback on Copilot code review. Take the survey.

[Alejandro-Vega]

LGTM! Everything still seems to be working with the change.