Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: group measurements by study #4617

Open
wants to merge 5 commits into
base: feat/measurements-service-filtering
Choose a base branch
from

Conversation

pedrokohler
Copy link
Collaborator

Context

Building on top of: #4501

Changes & Results

These changes group the measurements of the all-measurements tab by study.

Testing

Checklist

PR

  • [] My Pull Request title is descriptive, accurate and follows the
    semantic-release format and guidelines.

Code

  • [] My code has been well-documented (function documentation, inline comments,
    etc.)

Public Documentation Updates

  • [] The documentation page has been updated as necessary for any public API
    additions or removals.

Tested Environment

  • [] OS:
  • [] Node version:
  • [] Browser:

Copy link

netlify bot commented Dec 19, 2024

Deploy Preview for ohif-platform-docs ready!

Name Link
🔨 Latest commit a511a62
🔍 Latest deploy log https://app.netlify.com/sites/ohif-platform-docs/deploys/67643855b1f2e80008179325
😎 Deploy Preview https://deploy-preview-4617--ohif-platform-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 19, 2024

Deploy Preview for ohif-dev ready!

Name Link
🔨 Latest commit a511a62
🔍 Latest deploy log https://app.netlify.com/sites/ohif-dev/deploys/67643855de2b420009f8dae0
😎 Deploy Preview https://deploy-preview-4617--ohif-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pedrokohler pedrokohler changed the title Feat/measurements service filtering feat(measurementsPanel): group measurements by study Dec 19, 2024
@pedrokohler pedrokohler changed the title feat(measurementsPanel): group measurements by study feat: group measurements by study Dec 19, 2024
@wayfarer3130 wayfarer3130 changed the base branch from master to feat/measurements-service-filtering December 20, 2024 15:32
…/Viewers into feat/measurements-service-filtering
const oldValues = groupedMeasurements.get(key);
oldValues.push(item);
return groupedMeasurements;
}, new Map()) as Map<string, object[]>;
Copy link
Contributor

Choose a reason for hiding this comment

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

Cant you do new Map<string,object[]> to get the typing right?

{Array.from(measurements).map(([key, value]) => {
return (
<MeasurementTable
key={`tracked-${key}`}
Copy link
Contributor

@wayfarer3130 wayfarer3130 Dec 23, 2024

Choose a reason for hiding this comment

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

This isn't necessarily tracked on this version of it, so you shouldn't be referencing tracked here, but should just get something passed in from the parent about what you are naming this.

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