Skip to content

Kpmp 6545 biopsy tracking tableau#162

Merged
zwright merged 3 commits intodevelopfrom
KPMP-6545_Biopsy_Tracking_Tableau
Feb 4, 2026
Merged

Kpmp 6545 biopsy tracking tableau#162
zwright merged 3 commits intodevelopfrom
KPMP-6545_Biopsy_Tracking_Tableau

Conversation

@zwright
Copy link
Contributor

@zwright zwright commented Feb 4, 2026

Summary by CodeRabbit

  • New Features

    • Added new specimen tracking data field to biopsy records for enhanced data collection.
  • Chores

    • Improved data refresh process for biopsy tracking records to ensure data consistency.

@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

Walkthrough

The changes add a new dlu_tis column to the biopsy_tracking_long table schema and update the Tableau service to truncate the table before processing and include the new column in insert operations.

Changes

Cohort / File(s) Summary
Database Schema
data_management/sql/biopsy_tracking_long.sql
Added new column dlu_tis (varchar(100) NULL) to biopsy_tracking_long table positioned after specimen_id.
Tableau Service
data_management/services/tableau.py
Added truncate_biopsy_tracking_long() method and updated load_biopsy_tracking_long() to truncate table before processing and include dlu_tis in the insert query alongside redcap_id, specimen_id, dlu_packageType, and status.

Possibly related PRs

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch KPMP-6545_Biopsy_Tracking_Tableau

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

@zwright zwright merged commit 6d84115 into develop Feb 4, 2026
1 check passed
@zwright zwright deleted the KPMP-6545_Biopsy_Tracking_Tableau branch February 4, 2026 18:52
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