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

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

Closed
wants to merge 4 commits into from

Conversation

smentek
Copy link

@smentek smentek commented Jun 19, 2015

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

…lemented as switchable adapter with default value initialized düring object construction.
…lemented as switchable adapter with default value initialized during object construction.
@alexanderdean
Copy link
Member

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

@smentek
Copy link
Author

smentek commented Jun 19, 2015

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

@alexanderdean
Copy link
Member

Confirmed! Thanks so much

@smentek
Copy link
Author

smentek commented Jun 22, 2015

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
Copy link
Member

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
Copy link

skynet commented Aug 19, 2015

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
Copy link
Member

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
Copy link

skynet commented Aug 19, 2015

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

@ghost
Copy link

ghost commented Jan 3, 2018

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
Copy link
Member

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

@oguzhanunlu
Copy link
Member

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.

@oguzhanunlu oguzhanunlu removed this from the Version 0.3.0 milestone May 24, 2019
@adatzer adatzer closed this Feb 25, 2021
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

6 participants