Skip to content

Add external rcm viewer - #126

Merged
neuroelf merged 11 commits into
masterfrom
add-external-rcm-viewer
Jun 25, 2026
Merged

Add external rcm viewer#126
neuroelf merged 11 commits into
masterfrom
add-external-rcm-viewer

Conversation

@EkaterinaPoddubskaya

Copy link
Copy Markdown
Contributor

No description provided.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 16, 2026

Copy link
Copy Markdown

Deploying isic-gallery with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6a0a53d
Status: ✅  Deploy successful!
Preview URL: https://654db89f.isic-gallery.pages.dev
Branch Preview URL: https://add-external-rcm-viewer.isic-gallery.pages.dev

View logs

@EkaterinaPoddubskaya
EkaterinaPoddubskaya marked this pull request as ready for review June 18, 2026 15:42
@brianhelba
brianhelba removed their request for review June 18, 2026 16:17
@neuroelf

Copy link
Copy Markdown
Contributor

Hello @EkaterinaPoddubskaya -- I just clicked through one of the links, and saw that for lesion_id and patient_id, the "IL_" and "IP_" parts are missing. Would you mind making sure they are included in the URL parameters.

E.g., a current link looks like this:

https://rcm.dermannotator.org/?rcm_case_id=4070884&isic_id=ISIC_7726318&lesion_id=2268391&patient_id=5653891

And with the particles it would look like this:

https://rcm.dermannotator.org/?rcm_case_id=4070884&isic_id=ISIC_7726318&lesion_id=IL_2268391&patient_id=IP_5653891

Thank you!

@EkaterinaPoddubskaya

Copy link
Copy Markdown
Contributor Author

Hello @EkaterinaPoddubskaya -- I just clicked through one of the links, and saw that for lesion_id and patient_id, the "IL_" and "IP_" parts are missing. Would you mind making sure they are included in the URL parameters.

E.g., a current link looks like this:

https://rcm.dermannotator.org/?rcm_case_id=4070884&isic_id=ISIC_7726318&lesion_id=2268391&patient_id=5653891

And with the particles it would look like this:

https://rcm.dermannotator.org/?rcm_case_id=4070884&isic_id=ISIC_7726318&lesion_id=IL_2268391&patient_id=IP_5653891

Thank you!

Hello @neuroelf - I've added the "IL_" and "IP_" prefixes to the URL parameters. Let me know if you'd like any further changes :)

@neuroelf
neuroelf merged commit f8c0e7f into master Jun 25, 2026
3 checks passed
@neuroelf
neuroelf deleted the add-external-rcm-viewer branch June 25, 2026 17:35
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