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

Removing Gears detection, as its dead for a while now #334

Closed
wants to merge 21 commits into from
Closed

Removing Gears detection, as its dead for a while now #334

wants to merge 21 commits into from

Conversation

pebosi
Copy link
Contributor

@pebosi pebosi commented Jun 27, 2014

When accepting this change, i will update tests and provide an Update file to change DB Schema.

Please review

@mattab
Copy link
Member

mattab commented Jun 30, 2014

+1 for removing Gears. When the tests pass we will be happy to merge your pull request!

@peterbo
Copy link
Contributor

peterbo commented Jul 11, 2014

+1 for removing gears. But we have to be careful with this kind of changes - this will create a different recognition hash than before and affects every Piwik instance. This can lead to a corruption of metrics for a given recognition timeframe.

For the migration, we could use both hashes (with and without gears "0") but only store the new hash. This would not affect the quality of the metrics and could be completed in the next update (remove gears completely). Legacy code is bad, but the integrity of stats is very important.

@mattab
Copy link
Member

mattab commented Jul 15, 2014

This can lead to a corruption of metrics for a given recognition timeframe.

it's fine to change the recognition hash, for example I've changed it in 2.4.0 and it didn't worry me, because by default the hash is only used to match visits in the last 30 minutes though this can be changed

@mattab
Copy link
Member

mattab commented Aug 1, 2014

Dear @pebosi

Thank you for this proposed pull request.

Because it was last updated more than one month ago, it is our new policy to close pull requests opened for a long time without updates. If you would like to continue work on the pull request, please consider rebasing the PR to master and we can reopen it.

We hope you understand this and we look forward to seeing an update from you on this pull request or another one :)

Thanks.

@mattab mattab closed this Aug 1, 2014
@pebosi pebosi changed the base branch from master to 3.x-dev October 14, 2016 11:20
@pebosi
Copy link
Contributor Author

pebosi commented Oct 14, 2016

Changed base to 3.dev, re-open?

@mattab
Copy link
Member

mattab commented Nov 12, 2016

@pebosi I'd like to re-open but the repository where the code was has been deleted so I cannot reopen it directly. Would you be able to re-create the PR? would be great!

@pebosi
Copy link
Contributor Author

pebosi commented Mar 2, 2020

New try, see See #15627

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.

None yet

3 participants