Skip to content
This repository has been archived by the owner on May 26, 2019. It is now read-only.

Fixed header search-icon broken link (#2255) #2265

Closed
wants to merge 1 commit into from
Closed

Fixed header search-icon broken link (#2255) #2265

wants to merge 1 commit into from

Conversation

pakiusdevo
Copy link

No description provided.

@jenweber jenweber self-requested a review March 13, 2018 23:08
Copy link
Contributor

@jenweber jenweber left a comment

Choose a reason for hiding this comment

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

Hi @azanbertre, thanks for this! Can you edit to use a relative url instead? I think ../images/search-icon.svg might fix it. That way, no matter what version of the guides someone is looking at, the url still works.

@sivakumar-kailasam
Copy link
Member

Thanks for taking the time to contribute to this project. We’re in the process of switching from middleman to an ember app to make it easy for contribution.

We’re closing issue since its been open for too long without progress / it needs more discussion which is probably better had via a new PR in the new app. (sorry about that)

If this PR has changes to the content of guides please raise a PR to https://github.com/ember-learn/guides-source. If the change is to the guides viewer (i.e., the app) then please raise a PR in https://github.com/ember-learn/guides-app/. The READMEs and contributing.md guidelines of those projects will familiarize you with the new process. Let us know if you have any questions!

PS: this is a standard msg we're using across such PRs

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

Successfully merging this pull request may close these issues.

3 participants