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

Fixing hard coded links in HelpTranslations #749

Merged
merged 1 commit into from Jun 29, 2016

Conversation

lahmann
Copy link
Contributor

@lahmann lahmann commented Jun 29, 2016

Replaces all hard coded links in HelpTranslations with links build by view helper (otherwise links will fail if vufind is not installed in site root).

…by view helper (otherwise links will fail if vufind is not installed in site root)
@crhallberg
Copy link
Contributor

crhallberg commented Jun 29, 2016

Good catch! Thank you!

This also brings to my attention that these links needs data-lightbox-ignore so that they escape the lightbox. I'll take care of that after merging.

Hard-coded links were going to Search/Home, this fixed that problem. Thank you!

@crhallberg crhallberg merged commit f6859ab into vufind-org:master Jun 29, 2016
@lahmann
Copy link
Contributor Author

lahmann commented Jun 30, 2016

Hmm, I don't know if it is working for you now, but the route help-home does at least work for us.

@demiankatz
Copy link
Member

Thanks, Chris! Should some/all of this also go into the release-3.0 branch?

demiankatz pushed a commit that referenced this pull request Jul 5, 2016
…by view helper (otherwise links will fail if vufind is not installed in site root) (#749)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants