jrean / laravel-user-verification

PHP package built for Laravel to easily handle a user email verification and validate the email
Other
855 stars 118 forks source link

User Verification with Spark? #107

Closed gstoa closed 7 years ago

gstoa commented 7 years ago

Any success or how-to's leveraging this great package with Laravel 5.4 and Spark? Spark's registration process adds quite a bit of twists to the standard Laravel install, but I'd love to not have to write user email verification from scratch and use this package if possible.

ianfagg commented 7 years ago

I really need this too! Wish I was good enough to tackle it myself! May have to roll up my sleeves and try because I'm desperate. If you read this Taylor please give jrean a lifetime license for Spark if they solve it!

jrean commented 7 years ago

@ianfagg thank you for the feedback. I already sent an e-mail to Taylor to request a "test license" but he never replied.

ianfagg commented 7 years ago

Oh, that's a shame. I would have thought it was in his interest to do that, and costs him nothing!

I've bought Spark for my project and a bit miffed it does not have this feature. If I solve it based on your package I'll see if I can do a pull request (never done that before) to see if it can be added.

I'm definitely going to try your package for other projects too, it's a must have in my opinion. Thanks for sharing.

All the best, Ian

On 21 Apr 2017, at 11:31, Jean Ragouin notifications@github.com wrote:

@ianfagg https://github.com/ianfagg thank you for the feedback. I already sent an e-mail to Taylor to request a "test license" but he never replied.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jrean/laravel-user-verification/issues/107#issuecomment-296155977, or mute the thread https://github.com/notifications/unsubscribe-auth/AOI8eV_PDd4sPSylHPVxOHlmZz7Ne4-Lks5ryIVsgaJpZM4MJhlS.

ianfagg commented 7 years ago

I've almost got this working with Spark now, but where the documentation says:

Attributes/Properties To customize the package behaviour and the redirects you can implement and customize six (6) attributes/properties:

Sorry for my ignorance but where and how do I do that without overwriting the source files?

Thanks very much, Ian

jrean commented 7 years ago

Hi. Awesome! The attributes can be declared where you call the "traits". In the controller :-)

ianfagg commented 7 years ago

Thanks @jrean so I've added $this->redirectAfterVerification = '/login'; to the __construct of my RegisterController and that's working. However I'd like to be able to flash a message to the user to confirm they have been verified correctly. Any ideas how I could do that?

jrean commented 7 years ago

Sure! Use the event :-) UserVerified.php Create a listener and do what you want.

Jean

On May 4, 2017, at 3:34 PM, Ian notifications@github.com wrote:

Thanks @jrean so I've added $this->redirectAfterVerification = '/login'; to the __construct of my RegisterController and that's working. However I'd like to be able to flash a message to the user to confirm they have been verified correctly. Any ideas how I could do that?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

ianfagg commented 7 years ago

Of course, sorry, I'm an idiot!

I'll get onto that now, then I'll write up what I've had to do to get this working with Spark and forward it to you. I think it's pretty straight forward actually (although it's taken me days of trial and error because I don't know what I'm doing!).

Thanks again,

Ian

On 4 May 2017, at 10:45, Jean Ragouin notifications@github.com wrote:

Sure! Use the event :-) UserVerified.php Create a listener and do what you want.

Jean

On May 4, 2017, at 3:34 PM, Ian notifications@github.com wrote:

Thanks @jrean so I've added $this->redirectAfterVerification = '/login'; to the __construct of my RegisterController and that's working. However I'd like to be able to flash a message to the user to confirm they have been verified correctly. Any ideas how I could do that?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jrean/laravel-user-verification/issues/107#issuecomment-299128344, or mute the thread https://github.com/notifications/unsubscribe-auth/AOI8edFHV-HCfTKVkeT2Hb6zFxN7XjO8ks5r2ZAYgaJpZM4MJhlS.

jrean commented 7 years ago

It would be much appreciated :-) If you want go to https://askjong.com/contact/jong Drop me a message so we can exchange by e-mail.

On May 4, 2017, at 3:49 PM, Ian notifications@github.com wrote:

Of course, sorry, I'm an idiot!

I'll get onto that now, then I'll write up what I've had to do to get this working with Spark and forward it to you. I think it's pretty straight forward actually (although it's taken me days of trial and error because I don't know what I'm doing!).

Thanks again,

Ian

On 4 May 2017, at 10:45, Jean Ragouin notifications@github.com wrote:

Sure! Use the event :-) UserVerified.php Create a listener and do what you want.

Jean

On May 4, 2017, at 3:34 PM, Ian notifications@github.com wrote:

Thanks @jrean so I've added $this->redirectAfterVerification = '/login'; to the __construct of my RegisterController and that's working. However I'd like to be able to flash a message to the user to confirm they have been verified correctly. Any ideas how I could do that?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jrean/laravel-user-verification/issues/107#issuecomment-299128344, or mute the thread https://github.com/notifications/unsubscribe-auth/AOI8edFHV-HCfTKVkeT2Hb6zFxN7XjO8ks5r2ZAYgaJpZM4MJhlS.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

ianfagg commented 7 years ago

OK, will do.

On 4 May 2017, at 11:22, Jean Ragouin notifications@github.com wrote:

It would be much appreciated :-) If you want go to https://askjong.com/contact/jong Drop me a message so we can exchange by e-mail.

On May 4, 2017, at 3:49 PM, Ian notifications@github.com wrote:

Of course, sorry, I'm an idiot!

I'll get onto that now, then I'll write up what I've had to do to get this working with Spark and forward it to you. I think it's pretty straight forward actually (although it's taken me days of trial and error because I don't know what I'm doing!).

Thanks again,

Ian

On 4 May 2017, at 10:45, Jean Ragouin notifications@github.com wrote:

Sure! Use the event :-) UserVerified.php Create a listener and do what you want.

Jean

On May 4, 2017, at 3:34 PM, Ian notifications@github.com wrote:

Thanks @jrean so I've added $this->redirectAfterVerification = '/login'; to the __construct of my RegisterController and that's working. However I'd like to be able to flash a message to the user to confirm they have been verified correctly. Any ideas how I could do that?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jrean/laravel-user-verification/issues/107#issuecomment-299128344, or mute the thread https://github.com/notifications/unsubscribe-auth/AOI8edFHV-HCfTKVkeT2Hb6zFxN7XjO8ks5r2ZAYgaJpZM4MJhlS.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jrean/laravel-user-verification/issues/107#issuecomment-299135736, or mute the thread https://github.com/notifications/unsubscribe-auth/AOI8efaz_8ArKXKJ4ZsJ0wj7IcsEpkgNks5r2Zi6gaJpZM4MJhlS.

jrean commented 7 years ago

@ianfagg any news? :-)

jshannon63 commented 7 years ago

@jrean any word from @ianfagg yet on the spark implementation?

ianfagg commented 7 years ago

Sorry, been distracted by a couple of 'paid' projects. But hopefully get back onto it in the next week or so. I got it working as far as I remember, but was hoping to go live with the project to be sure. So need to check it through and write it up. I'll let you know...

ianfagg commented 7 years ago

OK, found my notes, take a look at this and see what you think... https://medium.com/@idff/laravel-spark-user-verification-75441cff5e44

franklincm commented 7 years ago

Happy to have found this. Thanks for the write-up!

jrean commented 7 years ago

Thank you @ianfagg I added a section in the documentation mentioning your article.