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.

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

New segment 'languageCode' #8085

Merged
merged 3 commits into from Jun 15, 2015
Merged

New segment 'languageCode' #8085

merged 3 commits into from Jun 15, 2015

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Jun 10, 2015

fixes #3950

@sgiehl sgiehl added the Needs Review PRs that need a code review label Jun 10, 2015
@mattab
Copy link
Member

mattab commented Jun 11, 2015

@sgiehl a couple System tests are failing, here is some background info:

@sgiehl sgiehl force-pushed the language_segment branch 2 times, most recently from e4b6939 to 62780d9 Compare June 12, 2015 16:27
@sgiehl
Copy link
Member Author

sgiehl commented Jun 12, 2015

@mattab system tests are ok now

@tsteur
Copy link
Member

tsteur commented Jun 15, 2015

Can you maybe try to add $dataTable->filter('AddSegmentValue'); similar as done here: https://github.com/piwik/piwik/blob/master/plugins/DevicesDetection/API.php#L237 in UserLanguage\API for those reports?

This might add the "One-click-segment feature" to those reports. If it works you will see in the language report on row hover an icon that opens a popup showing the visitor log with all the users having the same language. Unfortunately, we still have to add this manually each time. It's not guaranteed it works as easy as that though. If not, we could create a new issue to add it for those reports.

@sgiehl
Copy link
Member Author

sgiehl commented Jun 15, 2015

@tsteur seems to work

@tsteur
Copy link
Member

tsteur commented Jun 15, 2015

👍

tsteur added a commit that referenced this pull request Jun 15, 2015
@tsteur tsteur merged commit 5f289b4 into master Jun 15, 2015
@tsteur tsteur deleted the language_segment branch June 15, 2015 22:40
@mattab
Copy link
Member

mattab commented Jun 16, 2015

@sgiehl I found a minor issue, see previous comment.

otherwise 👍

@sgiehl
Copy link
Member Author

sgiehl commented Jun 16, 2015

Ok. I'll have a look

@sgiehl
Copy link
Member Author

sgiehl commented Jun 16, 2015

I've checked that and push it to master later. Correct segment is languageCode==fr,languageCode=@fr- to avoid matches for language codes like de-fr

sgiehl added a commit that referenced this pull request Jun 16, 2015
@mattab
Copy link
Member

mattab commented Jun 17, 2015

Correct segment is languageCode==fr,languageCode=@fr- to avoid matches for language codes like de-fr

oh that's smart! 👍

@mgonera
Copy link

mgonera commented Jun 25, 2015

@mattab @sgiehl Please confirm if this will be included in 2.14 or in which other version?

@sgiehl
Copy link
Member Author

sgiehl commented Jun 25, 2015 via email

@mattab mattab added this to the 2.14.0 milestone Jun 25, 2015
@mattab
Copy link
Member

mattab commented Jun 25, 2015

@mgonera Here is a useful github trick. if you click on the commit, you will arrive on commit page. there Github shows which tag already contains this commit. For this particular commit you can see:
commit

-> this change is available since 2.14.0-b4

@mgonera
Copy link

mgonera commented Jun 26, 2015

@mattab very cool, thanks! this for sure will come in handy

@mattab mattab changed the title New segment 'language' New segment 'languageCode' Jun 26, 2015
@mattab
Copy link
Member

mattab commented Jun 26, 2015

Note: the new segment is actually languageCode. Added to developer changelog in 980c12b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants