Showing how to access specific data items #3468

Merged
merged 6 commits into from Apr 10, 2015

Conversation

Projects
None yet
3 participants
@davidsilvasmith
Contributor

davidsilvasmith commented Feb 15, 2015

This patch is an update to the docs showing how to access specific data items via front matter variables.

@parkr parkr added the Documentation label Feb 16, 2015

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Mar 8, 2015

Member

This has your LSI docs update as well :(

Member

parkr commented Mar 8, 2015

This has your LSI docs update as well :(

+author: dave
+---
+
+{% assign author = site.data.people.[page.author] %}

This comment has been minimized.

@parkr

parkr Mar 8, 2015

Member

Isn't the . after people invalid?

@parkr

parkr Mar 8, 2015

Member

Isn't the . after people invalid?

This comment has been minimized.

@davidsilvasmith

davidsilvasmith Mar 10, 2015

Contributor

It's a weird syntax (super useful though, took me some googling to get it). I confirmed it's working correctly on my site: https://github.com/davidsilvasmith/bitcoinbulls.net/blob/gh-pages/_layouts/post.html

Sorry about including the LSI docs. I'm new to pull requests. I'll be better with future pulls :)

@davidsilvasmith

davidsilvasmith Mar 10, 2015

Contributor

It's a weird syntax (super useful though, took me some googling to get it). I confirmed it's working correctly on my site: https://github.com/davidsilvasmith/bitcoinbulls.net/blob/gh-pages/_layouts/post.html

Sorry about including the LSI docs. I'm new to pull requests. I'll be better with future pulls :)

This comment has been minimized.

@parkr

parkr Mar 26, 2015

Member

Does it work without the added dot? It will likely break in Liquid 3's stricter lexer.

@parkr

parkr Mar 26, 2015

Member

Does it work without the added dot? It will likely break in Liquid 3's stricter lexer.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Mar 8, 2015

Member

One comment, then I'll merge! Thanks, Dave!

Member

parkr commented Mar 8, 2015

One comment, then I'll merge! Thanks, Dave!

@parkr parkr merged commit e9638fa into jekyll:master Apr 10, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

parkr added a commit that referenced this pull request Apr 10, 2015

@davidsilvasmith davidsilvasmith deleted the davidsilvasmith:origin/patch-3 branch Aug 4, 2015

@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.