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: Fix links to extension icon and screenshots #1

Closed
wants to merge 1 commit into from

Conversation

jamesobutler
Copy link

This fixes invalid icon links causing no image to appear for the extension in Slicer's Extension Manager.

image

cc: @acsenrafilho

@jamesobutler
Copy link
Author

@acsenrafilho Can you help with the integration of this PR? Thanks!

jamesobutler added a commit to jamesobutler/ExtensionsIndex that referenced this pull request Jul 6, 2022
We can switch back to the original repository when this pull request is merged:
CSIM-Toolkits/SlicerBrainVolumeRefinement#1
jamesobutler added a commit to jamesobutler/ExtensionsIndex that referenced this pull request Jul 6, 2022
We can switch back to the original repository when this pull request is merged:
CSIM-Toolkits/SlicerBrainVolumeRefinement#1
lassoan pushed a commit to Slicer/ExtensionsIndex that referenced this pull request Jul 6, 2022
We can switch back to the original repository when this pull request is merged:
CSIM-Toolkits/SlicerBrainVolumeRefinement#1
lassoan pushed a commit to Slicer/ExtensionsIndex that referenced this pull request Jul 6, 2022
We can switch back to the original repository when this pull request is merged:
CSIM-Toolkits/SlicerBrainVolumeRefinement#1
@acsenrafilho
Copy link
Member

@jamesobutler

Sorry for the long delay here.
I had made the changes in the repo to allow the correct image links.

Iǘe sent a new PR to Slicer ExtensionIndex repo to fix this.

Sorry again for the inconvenience. I will close this PR since it's already fixed by your fork.

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