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

Clarify how to access site #1864

Merged
merged 1 commit into from
Dec 23, 2013
Merged

Clarify how to access site #1864

merged 1 commit into from
Dec 23, 2013

Conversation

pootsbook
Copy link
Contributor

I was unable to access the config at context.registers.config. Everything seems to point towards the site variable being accessed via context.registers[:site].

I was unable to access the config at `context.registers.config`. Everything seems to point towards the `site` variable being accessed via `context.registers[:site]`.
@parkr
Copy link
Member

parkr commented Dec 23, 2013

👍 Thanks!

parkr added a commit that referenced this pull request Dec 23, 2013
@parkr parkr merged commit 3d09b14 into jekyll:master Dec 23, 2013
parkr added a commit that referenced this pull request Dec 23, 2013
@@ -339,9 +339,9 @@ Liquid::Template.register_filter(Jekyll::AssetFilter)
<h5>ProTip™: Access the site object using Liquid</h5>
<p>
Jekyll lets you access the <code>site</code> object through the
<code>context.registers</code> feature of Liquid. For example, you can
<code>context.registers</code> feature of Liquid at <code>context.registers[:site]. For example, you can
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing a </code> end tag here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good spot. I think this PR has been merged. I'll open another.

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
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.

None yet

4 participants