Skip to content
This repository was archived by the owner on Sep 4, 2018. It is now read-only.

Conversation

@koke
Copy link
Member

@koke koke commented Nov 10, 2015

Sets header_dir to WordPressComAnalytics to avoid problems with
underscores/dashes in names.

I'd bump the minor version since this breaks backwards compatibility,
but I'm not sure what to do since it's 0.0

See wordpress-mobile/WordPress-iOS#4434

Needs Review: @kwonye

Sets header_dir to WordPressComAnalytics to avoid problems with
underscores/dashes in names.

I'd bump the minor version since this breaks backwards compatibility,
but I'm not sure what to do since it's 0.0
@koke
Copy link
Member Author

koke commented Nov 11, 2015

Merged develop, ready for review

@koke
Copy link
Member Author

koke commented Nov 11, 2015

Found an issue with bundles and the data model, don't merge yet

@koke
Copy link
Member Author

koke commented Nov 11, 2015

Never mind, I keep confusing Analytics with Tracks, problem is with a different pod 😫

kwonye added a commit that referenced this pull request Nov 12, 2015
@kwonye kwonye merged commit 4f0daf8 into develop Nov 12, 2015
@kwonye
Copy link
Contributor

kwonye commented Nov 12, 2015

Whoops, sorry @koke I forgot the iOS side prefers to merge in themselves. Looked great though :shipit:

@kwonye kwonye deleted the fix/framework-support branch November 12, 2015 02:49
This was referenced Nov 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants