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 #239

Merged
merged 2 commits into from
Feb 7, 2016
Merged

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

merged 2 commits into from
Feb 7, 2016

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

Changed: target to dev branch

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

Hey @kylewelsby I intend to look closer at this sometime this week or this weekend.
Thanks for your contribution! Poke me here or on Twitter @bmuenzenmeyer if I forget

@kylewelsby
Copy link
Contributor Author

thanks, no problem

@kylewelsby
Copy link
Contributor Author

I have just pushed another bugfix to this issue, it basically enables pseudo-patterns to be linked to within data.

bmuenzenmeyer pushed a commit that referenced this pull request Feb 7, 2016
bugfix: #171 pattern links inside base data object
@bmuenzenmeyer bmuenzenmeyer merged commit f187cf4 into pattern-lab:dev Feb 7, 2016
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.

2 participants