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

Region helper MERC-2432 #119

Closed

Conversation

tabayomi
Copy link
Contributor

@tabayomi tabayomi commented Jun 5, 2017

What:
This PR extends Paper by adding a location helper for content blocks. This enables access inside the helper to using all the available helpers for rendering inside Paper.

Depends on:
#118.

06/14 -- Updated names to Region based on new content service nomenclature

Copy link
Contributor

@mcampa mcampa left a comment

Choose a reason for hiding this comment

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

Please add a test. All helpers have tests

let locationId = params.hash.name;
let content = '';

if ((!paper.renderingContext) || (!paper.renderingContext.locations)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

extra parenthesis

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure where you're referring..

Copy link
Contributor

Choose a reason for hiding this comment

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

if (!paper.renderingContext || !paper.renderingContext.locations) {

@tabayomi tabayomi force-pushed the location-helper-MERC-2432 branch 4 times, most recently from 351ef8f to 5764ebb Compare June 8, 2017 18:07
@tabayomi tabayomi changed the title Location helper MERC-2432 Region helper MERC-2432 Jun 14, 2017
@tabayomi
Copy link
Contributor Author

Merging the changes in this PR together with #118

Closing this.

@tabayomi tabayomi closed this Jun 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants