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

Adds values for Account Settings/Profile#45

Merged
koke merged 1 commit intodevelopfrom
feature/profile-account-settings
Dec 4, 2015
Merged

Adds values for Account Settings/Profile#45
koke merged 1 commit intodevelopfrom
feature/profile-account-settings

Conversation

@koke
Copy link
Copy Markdown
Member

@koke koke commented Nov 25, 2015

It seems WPiOS was using WPAnalyticsStatOpenedSettings for both site
settings and account settings. Renamed it for clearer distinction.

For wordpress-mobile/WordPress-iOS#4510

Needs Review: @astralbodies

It seems WPiOS was using WPAnalyticsStatOpenedSettings for both site
settings and account settings. Renamed it for clearer distinction.
@koke
Copy link
Copy Markdown
Member Author

koke commented Dec 4, 2015

Somehow forgot to ping someone for review :(

@astralbodies
Copy link
Copy Markdown
Contributor

Ooops!

:shipit:

@koke Can you make sure a similar change is logged for the Android side? cc: @daniloercoli

@oguzkocer
Copy link
Copy Markdown
Contributor

@astralbodies I can get that added once the feature is finished in Android side. Thanks!

koke added a commit that referenced this pull request Dec 4, 2015
…settings

Adds values for Account Settings/Profile
@koke koke merged commit 88a5889 into develop Dec 4, 2015
@koke koke deleted the feature/profile-account-settings branch December 4, 2015 15:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants