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 broken image links #253

Merged
merged 1 commit into from
Oct 8, 2022
Merged

Conversation

jakeoeding
Copy link
Contributor

There were several broken links. They all pointed to a develop branch in the nord-docs repo, which seems to no longer exist. The paths were updated to point to the images on the main branch instead.

There were several broken links. They all pointed to a `develop`
branch in the nord-docs repo, which seems to no longer exist. The
paths were updated to point to the images on the `main` branch
instead.
Copy link
Member

@svengreb svengreb left a comment

Choose a reason for hiding this comment

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

Hi @jakeoeding 👋, thanks for your contribution 👍

This is caused by the recent GitHub flow migration in the nord-docs repository as well as the switch from the master to main branch. This is also in preparation for a very big change that is scheduled to be announced in a few weeks which require the paths to be updated again, but in the meantime these should definitely be fixed in almost all Nord port repositories.

@svengreb svengreb merged commit 1cfe0d0 into nordtheme:develop Oct 8, 2022
@svengreb svengreb removed their assignment Oct 8, 2022
@aryadaroui
Copy link

This update is not reflected in the vs code / microsoft marketplace. the image links are still broken and pointing to the old develop branch

https://marketplace.visualstudio.com/items?itemName=arcticicestudio.nord-visual-studio-code

Screenshot 2023-01-08 at 10 25 36 PM

@jakeoeding
Copy link
Contributor Author

@svengreb I think a publish may be needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants