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

[Bug] No way to show only primary study in left side Studies List by default. #4530

Closed
ranasrule opened this issue Nov 19, 2024 · 4 comments
Closed
Assignees

Comments

@ranasrule
Copy link

ranasrule commented Nov 19, 2024

Describe the Bug

There should be some way to show only primary study in left side Studies List by default. Currently all of the patients studies are shown by default.
386817256-9fdf8ec2-5305-4bfd-b8d7-9324c0a54b32

Steps to Reproduce

1

The current behavior

all of the patient's studies are shown

The expected behavior

some way to show only primary study

OS

win 11

Node version

18.16

Browser

chrome

@ranasrule ranasrule added the Awaiting Reproduction Can we reproduce the reported bug? label Nov 19, 2024
@sedghi
Copy link
Member

sedghi commented Nov 19, 2024

I think we should add a config, you are right

@sedghi sedghi removed the Awaiting Reproduction Can we reproduce the reported bug? label Nov 19, 2024
@sedghi sedghi assigned sedghi and unassigned IbrahimCSAE Nov 25, 2024
@sedghi sedghi closed this as completed Dec 3, 2024
@ranasrule
Copy link
Author

@sedghi I see that this has been marked as closed. How can we now show only primary study in left side Studies List?

@IbrahimCSAE
Copy link
Collaborator

IbrahimCSAE commented Dec 6, 2024

@sedghi I see that this has been marked as closed. How can we now show only primary study in left side Studies List?

See here:

const { mode: studyMode } = customizationService.getCustomization('PanelStudyBrowser.studyMode', {

Use the customizationService and set mode to primary @ranasrule

@ranasrule
Copy link
Author

@sedghi I see that this has been marked as closed. How can we now show only primary study in left side Studies List?

See here:

Viewers/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.tsx

Line 49 in 095ab7e
const { mode: studyMode } = customizationService.getCustomization('PanelStudyBrowser.studyMode', {

Use the customizationService and set mode to primary @ranasrule

Thanks for your reply @IbrahimCSAE

so I simply change

const { mode: studyMode } = customizationService.getCustomization('PanelStudyBrowser.studyMode', {
    id: 'default',
    mode: 'all',
  });

to

const { mode: studyMode } = customizationService.getCustomization('PanelStudyBrowser.studyMode', {
    id: 'default',
    mode: 'primary',
  });

and then build ohif ?

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

No branches or pull requests

3 participants