Skip to content

Conversation

@cnovak
Copy link
Collaborator

@cnovak cnovak commented Sep 7, 2017

Company app analytics was not working properly because the call to analytics was filtering by developer. Removed filter from DeveloperAppAnalytics.getByAppName that was filtering by developer.

Company app analytics was not working properly because the call to analytics was filtering by developer. Removed filter from DeveloperAppAnalytics.getByAppName that was filtering by developer.
@ossobuffo
Copy link
Contributor

ossobuffo commented Sep 7, 2017

Watch out. If you do not filter by developer, if you query test-app it will return results for all apps for all developers by that name. I suggest instead altering that filter if the app is a company app. You will need to ask the analytics people how to structure your filter for a company name.

@cnovak
Copy link
Collaborator Author

cnovak commented Sep 7, 2017

Thanks @ossobuffo ! I thought there was a reason that filter was there, but could not think of why. I will close this PR and look for a way to filter by company. I do not think it is possible however, since Edge analytics screens say "AppName by (not set)" when it is a company app, where "not set" should be the developer's name.

@cnovak cnovak closed this Sep 7, 2017
@cnovak cnovak deleted the 65101827 branch September 8, 2017 22:33
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