Skip to content

Conversation

@slorber
Copy link
Collaborator

@slorber slorber commented Jul 31, 2018

@alvaromb it's a work in progress, not to merge yet

if you have any opinion on this tell me please

@slorber slorber requested a review from alvaromb August 1, 2018 13:55
@slorber
Copy link
Collaborator Author

slorber commented Aug 1, 2018

hey @alvaromb , my code seems to work fine in my app

I am not able to merge my PR, not to push directly on master

@slorber
Copy link
Collaborator Author

slorber commented Aug 3, 2018

@alvaromb can you give me the rights to merge and publish? Or approve the PR?

@alvaromb
Copy link
Collaborator

alvaromb commented Aug 6, 2018

Sorry @slorber, please try again

@slorber
Copy link
Collaborator Author

slorber commented Aug 6, 2018

thanks, this seems to be working now. I'll make a release with notes soon

@slorber slorber merged commit 98f4912 into APSL:master Aug 6, 2018
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