Skip to content

Adds missing xml attributes#392

Merged
cammace merged 4 commits intomasterfrom
fix-public-attr
Apr 2, 2018
Merged

Adds missing xml attributes#392
cammace merged 4 commits intomasterfrom
fix-public-attr

Conversation

@cammace
Copy link
Copy Markdown

@cammace cammace commented Mar 29, 2018

closes #368

Adds new attributes and fixes private style.

@cammace cammace added the ready for review When your PR has been personally reviewed, its time for an external contributors to approve label Mar 29, 2018
@cammace cammace added this to the location-layer-0.5.0 milestone Mar 29, 2018
@cammace cammace self-assigned this Mar 29, 2018
@cammace cammace requested review from danesfeder and tobrun March 29, 2018 21:25
Copy link
Copy Markdown
Contributor

@danesfeder danesfeder left a comment

Choose a reason for hiding this comment

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

Looks good, just one typo (I think). Thanks @cammace

* attributes while leaving the rest alone and maintaining their default behavior.
*
* @param context your activites context used to acquire the style resource
* @param context your activates context used to acquire the style resource
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you meant to fix to activities here?

@cammace cammace merged commit 52922a4 into master Apr 2, 2018
@cammace cammace deleted the fix-public-attr branch April 2, 2018 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review When your PR has been personally reviewed, its time for an external contributors to approve

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make all location layer attributes public

3 participants