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

fix(tools): check if seriesNumber is an undefined properly #4338

Merged
merged 5 commits into from
Oct 17, 2024

Conversation

IbrahimCSAE
Copy link
Collaborator

Context

some tools were not rendering displayText if series number is 0, now we check properly for undefined.

Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for ohif-platform-docs canceled.

Name Link
🔨 Latest commit 04962dd
🔍 Latest deploy log https://app.netlify.com/sites/ohif-platform-docs/deploys/66bf651b9bc1c20007bcd7b6

Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for ohif-dev canceled.

Name Link
🔨 Latest commit 04962dd
🔍 Latest deploy log https://app.netlify.com/sites/ohif-dev/deploys/66bf651bab76b70008a378bc

Copy link

cypress bot commented Aug 16, 2024



Test summary

44 0 2 0


Run details

Project Viewers
Status Passed
Commit 04962dd
Started Aug 16, 2024 2:57 PM
Ended Aug 16, 2024 2:59 PM
Duration 02:39 💡
OS Linux Ubuntu -
Browser Chrome 127

View run in Cypress Cloud ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud

This reverts commit ea435fe.
This reverts commit 455b4d3.
@sedghi sedghi merged commit 307b144 into master Oct 17, 2024
14 checks passed
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