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

bugfix: #171 pattern links inside base data object #238

Closed
wants to merge 1 commit into from
Closed

bugfix: #171 pattern links inside base data object #238

wants to merge 1 commit into from

Conversation

kylewelsby
Copy link
Contributor

Hey, I would like to submit a minor bugfix to v1.0.0 regarding Added ability to specify link. urls inside data objects*

This minor refactor allows for pattern links insde _data.json as well as pattern *.json objects

Also notice the RegExp has been updated to only match link\.[A-z0-9-_] to avoid matching link123something or link*something

This minor refactor allows for pattern links insde `_data.json` as well as pattern `*.json` objects

Also notice the RegExp has been updated to only match `link\.[A-z0-9-_]` to avoid matching `link123something` or `link*something`
@bmuenzenmeyer
Copy link
Member

this is great @kylewelsby thanks so much!

Can you please repoint this at the dev branch? you shouldn't have to create a new PR, just edit this one

@kylewelsby kylewelsby closed this Jan 29, 2016
@kylewelsby
Copy link
Contributor Author

have changed the target in #239

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

Successfully merging this pull request may close these issues.

None yet

2 participants