-
Notifications
You must be signed in to change notification settings - Fork 506
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
shadowhand/email deprecated and removed, breaking setup #1626
Comments
Confirmed on my side. Since it's probably affecting everyone (including our own deployment processes), I think this deserves P0 @jshorland |
Yep. Add p0 and put in dev backlog please?
iPhone iTypo
… On Mar 6, 2017, at 5:21 PM, David Losada ***@***.***> wrote:
Confirmed on my side. Since it's probably affecting everyone, I think this deserves P0 @jshorland
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I found this module after digging through the internet, https://github.com/kierangraham/email |
@sudheesh001 if you have time and would like to contribute we'd be happy to receive said help :D If you want to submit a PR you can assign me for review and I'll take a look. |
@willdoran I've still got a cached copy of the module I can drop in a repo or compare against the keirangraham one. |
@rjmackay awesome, that would be super handy, if you link me I can compare |
@willdoran https://github.com/kierangraham/email is the original kohana email module, rather than the updated version packaged for composer. I've pushed my cached version to https://github.com/rjmackay/shadowhand-email |
@rjmackay Awesome ! Have you uploaded the entry to packagist so that we can update the composer entry ? |
@rjmackay I've updated my composer file to contain a reference to https://packagist.org/packages/sudheesh001/email for the breaking |
@willdoran did you have time to resolve this? if not I'll take a look this week |
@rjmackay No, I didn't get a chance yet. If I do before you're online. I'll update here. |
@sudheesh001 I've fixed this on the master and develop branches now. It uses ushahidi/shadowhand-email |
Having reported this a couple of days ago via the developer community (www.hipchat.com?), I can confirm this is now fixed for me (though I've now hit another problem #1654). |
Looks like this is fixed, good to close. 👍 |
Actual behaviour
The setup process involves the
composer
process fetching the required modules but it looks like theshadowhand/email
module has been removed altogether by the owner after deprecation. ushahidi is still dependant on the module, is there an alternative module that we could use to set this up ?Expected behaviour
A clean setup process after
bin/update
Steps to reproduce the behaviour/error
Run
bin/update
The text was updated successfully, but these errors were encountered: