Skip to content

[DT-2855] Add optional Through.Bio ID field to Study Registration form#3330

Open
kevinmarete wants to merge 7 commits intodevelopfrom
km_DT-2855_add_through_bio_field_to_form
Open

[DT-2855] Add optional Through.Bio ID field to Study Registration form#3330
kevinmarete wants to merge 7 commits intodevelopfrom
km_DT-2855_add_through_bio_field_to_form

Conversation

@kevinmarete
Copy link
Contributor

@kevinmarete kevinmarete commented Feb 17, 2026

Addresses

https://broadworkbench.atlassian.net/browse/DT-2855

Summary

Adds a new optional Through.Bio ID field to the bottom of the Study section in the Study Registration form. Here is the backend PR DataBiosphere/consent#2816

  • Displays static prefix text: “Through.bio/”
  • Persists the identifier as part of the Study record
  • Field is optional (no validation error when empty)
  • Accepts identifier only (full URLs are not allowed)
After.mov

Have you read Terra's Contributing Guide lately? If not, do that first.

  • Label PR with a Jira ticket number and include a link to the ticket
  • Label PR with a security risk modifier [no, low, medium, high]
  • PR describes scope of changes
  • Get a minimum of one thumbs worth of review, preferably two if enough team members are available
  • Get PO sign-off for all non-trivial UI or workflow changes
  • Verify all tests go green
  • Test this change deployed correctly and works on dev environment after deployment

@kevinmarete kevinmarete self-assigned this Feb 17, 2026
@kevinmarete kevinmarete changed the title [DT-2855] Add Through.bio field to Data Submission Form [DT-2855] Add optional Through.Bio ID field to Study Registration form Feb 17, 2026
@kevinmarete kevinmarete changed the title [DT-2855] Add optional Through.Bio ID field to Study Registration form [DT-2855] Add optional Through.Bio ID field to Study Registration form Feb 17, 2026
@kevinmarete kevinmarete marked this pull request as ready for review February 17, 2026 23:38
@kevinmarete kevinmarete requested a review from a team as a code owner February 17, 2026 23:38
@kevinmarete kevinmarete requested review from fboulnois and rushtong and removed request for a team February 17, 2026 23:38
Copy link
Contributor

@rushtong rushtong left a comment

Choose a reason for hiding this comment

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

The UI side looks good, but I don't see this persisted on the back end. I created study ID 9504 in dev with a through.bio value but it doesn't seem to be there when being reloaded into the edit study form.

@kevinmarete
Copy link
Contributor Author

The UI side looks good, but I don't see this persisted on the back end. I created study ID 9504 in dev with a through.bio value but it doesn't seem to be there when being reloaded into the edit study form.

Forgot to mention the backend has not been implemented yet. This is the follow-up ticket for backend: https://broadworkbench.atlassian.net/browse/DT-2882

@kevinmarete kevinmarete requested a review from rushtong February 18, 2026 15:30
Copy link
Contributor

@rushtong rushtong left a comment

Choose a reason for hiding this comment

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

👍🏽

Copy link
Contributor

@fboulnois fboulnois left a comment

Choose a reason for hiding this comment

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

Ditto Greg's question, otherwise:

@sonarqubecloud
Copy link

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.

3 participants

Comments