Skip to content

Conversation

@dmendelowitz
Copy link
Contributor

Summary

stagingTemplate: The profile TNMStageGroup is now renamed CancerStageGroup in order to support non-TNM staging systems such as Rai, Binet, and Revised International Staging System (R-ISS)

tnmCategoryTemplate: In STU2, the separate sets of profiles for TNM Clinical and TNM Pathologic staging were combined into a single set of profiles: CancerStageGroup, TNMPrimaryTumorCategory, TNMRegionalNodesCategory, and TNMDistantMetastasesCategory.

New behavior

All profiles that used to distinguish between Clinical and Pathologic have been consolidated to single profiles, but staging type is still necessary as an input because it is used to determine code and category within those profiles.

Code changes

  • stagingTemplate.js now uses the new Cancer Stage Group profile
  • TNMCategoryTemplate.js now uses the new single set of profiles for Tumor, Nodes, and Metastases
  • All relevant test fixtures have been updated to reflect changes

Testing guidance

  • Make sure all tests still pass
  • Run the client with the Staging Extractor and see that the new profiles from STU2 are used
  • Let me know if there are any changes from STU2 that I missed or implemented incorrectly

Copy link
Contributor

@Dtphelan1 Dtphelan1 left a comment

Choose a reason for hiding this comment

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

This looks great. Can confirm tests still pass, double-checked the new resource profiles and there aren't any features mentioned in the Diff Table or the Change Log that we don't have covered. Great work!

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