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

sync with ohif viewer #2512

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

thanh-nguyen-dang
Copy link
Contributor

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/MIDRC-639

New Features

  • Update new port for OHIF Viewer after sync with latest version in OHIF

Breaking Changes

Bug Fixes

  • Update new port for OHIF Viewer

Improvements

Dependency updates

Deployment changes

Copy link
Contributor

@paulineribeyre paulineribeyre left a comment

Choose a reason for hiding this comment

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

I suspect the currently deployed version of the DICOM viewer will break if we merge this

We'll have to wait until we're ready to deploy the new version to prod

@thanh-nguyen-dang
Copy link
Contributor Author

I suspect the currently deployed version of the DICOM viewer will break if we merge this

We'll have to wait until we're ready to deploy the new version to prod

yeah, this PR is intended to use for the new version @paulineribeyre

@@ -7,7 +7,7 @@ spec:
app: dicom-viewer
ports:
- protocol: TCP
port: 80
port: 8080
Copy link
Contributor

Choose a reason for hiding this comment

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

This should still be port 80, but target port should be 8080

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks @jawadqur

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