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

Doc language fix #1062

Merged
merged 2 commits into from Jul 1, 2013
Merged

Doc language fix #1062

merged 2 commits into from Jul 1, 2013

Conversation

tobes
Copy link
Contributor

@tobes tobes commented Jul 1, 2013

wrong url for api docs with none base languages eg en_GB pt_BR

@ghost ghost assigned tobes Jul 1, 2013
@ghost ghost assigned nigelbabu Jul 1, 2013
@nigelbabu
Copy link
Contributor

Looks good to me.

nigelbabu added a commit that referenced this pull request Jul 1, 2013
@nigelbabu nigelbabu merged commit 7e59901 into master Jul 1, 2013
@nigelbabu nigelbabu deleted the 1062-doc-lang-fix branch July 1, 2013 12:55
@nigelbabu
Copy link
Contributor

@tobes The url generated by this code does not actually work, which I missed. Is it better to force "en" for now or do we want to put CKAN_LANG_BASE in the environment?

@tobes
Copy link
Contributor Author

tobes commented Jul 1, 2013

@nigelbabu sorry I'll fix now

tobes added a commit that referenced this pull request Jul 1, 2013
@tobes
Copy link
Contributor Author

tobes commented Jul 1, 2013

I added this to master but needed for 2.1

tobes added a commit that referenced this pull request Jul 1, 2013
Conflicts:

	ckan/templates/admin/config.html
	ckan/templates/admin/index.html
	ckan/templates/footer.html
	ckan/templates/package/search.html
tobes added a commit that referenced this pull request Jul 1, 2013
Conflicts:

	ckan/templates/admin/config.html
	ckan/templates/admin/index.html
	ckan/templates/footer.html
	ckan/templates/package/search.html
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.

None yet

3 participants