search

snowplow / snowplow-php-tracker

Snowplow event tracker for PHP. Add analytics into your PHP apps and scripts
http://snowplowanalytics.com
34 stars 36 forks source link

Switch dependency for hash generation to: webparser/laravel-uuid. #53

Closed smentek closed 3 years ago

smentek commented 9 years ago

Implemented as switchable adapter with default value initialized during object construction.

alexanderdean commented 9 years ago

I like it! Can you sign our CLA please: https://github.com/snowplow/snowplow/wiki/CLA

smentek commented 9 years ago

I'm happy you like it. I've signed "Software Grant and Corporate Contributor License Agreement v1.0" for you.

alexanderdean commented 9 years ago

Confirmed! Thanks so much

smentek commented 9 years ago

Could you please give time estimation when new release with #53 will be created? We need to deploy, and If we know that you need more time I will go with forked version until official is ready...

alexanderdean commented 9 years ago

Hi @smentek - I don't have a time estimate now but I can tell you it's low priority in our unplanned work queue, so yes please go ahead and deploy with your forked version till ready.

skynet commented 9 years ago

is this going to make it to the master soon? I get a warning from composer: 

Package rhumsaa/uuid is abandoned, you should avoid using it. Use ramsey/uuid instead.
alexanderdean commented 9 years ago

Hey @skynet - thanks for the heads-up. We don't have a timeline on this yet but we will try and get to this soon.

skynet commented 9 years ago

I wonder if I can just work with @smentek 's fork in the meantime. Thanks.

ghost commented 6 years ago

Hi @alexanderdean do you have any update on this? We are having issues with 

Package rhumsaa/uuid is abandoned, you should avoid using it. Use ramsey/uuid instead.

Hope this will get to the new release soon,

Thanks

alexanderdean commented 6 years ago

Have you tried using the 0.3.0-rc1 @0x13a?

oguzhanunlu commented 5 years ago

Hi @smentek , 0.3.0 will be finalized soon and a switch from rhumsaa/uuid to ramsey/uuid is included there, see #75 . Please renew your PR if you're still interested in this change.