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.

Sign up for GitHub

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

Jump to bottom

Supporting segmentation for Countly #209

Merged
merged 1 commit into from
Sep 21, 2015
Merged

Supporting segmentation for Countly #209

merged 1 commit into from
Sep 21, 2015

Conversation

gugmaster
Copy link
Contributor

I only changed to segmentation support in Countly.

Thanks for all your great work.
Best regards,
Stefan

@orta
Copy link
Owner

orta commented Sep 11, 2015

will this handle nil properties fine?

@gugmaster
Copy link
Contributor Author

Good point, did not think on this, but yes it works (tested now).

On 11 Sep 2015, at 15:09, Orta notifications@github.com wrote:

will this handle nil properties fine?

—
Reply to this email directly or view it on GitHub #209 (comment).

@orta
Copy link
Owner

orta commented Sep 21, 2015

Ace, thanks!

orta added a commit that referenced this pull request Sep 21, 2015
@orta
Merge pull request #209 from gugmaster/gugmaster-countly-segmentation
724d33e 
Supporting segmentation for Countly
@orta orta merged commit 724d33e into orta:master Sep 21, 2015
@gugmaster
Copy link
Contributor Author

You’re welcome 😉

On 21 Sep 2015, at 14:38, Orta notifications@github.com wrote:

Ace, thanks!

—
Reply to this email directly or view it on GitHub #209 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gugmaster @orta