hanami / ecosystem

Hanami Ecosystem
12 stars 3 forks source link

Add basic attempt at an authentication doc; ref #5 #7

Closed greggilbert closed 7 years ago

greggilbert commented 7 years ago

Just a quick doc based on my rudimentary setup of Warden. Almost certainly room for improvement, but it's a start.

davydovanton commented 7 years ago

Thanks for contribution 👍 This PR looks good for me. But I think we need to change sequel to hanami-model in this example.

What do you think @cllns?

AlfonsoUceda commented 7 years ago

as @davydovanton said, we should use Hanami::Model instead Sequel. @greggilbert thans for the PR but could you update with hanami-model?

AlfonsoUceda commented 7 years ago

ping @greggilbert

cllns commented 7 years ago

@AlfonsoUceda, @davydovanton and I are working on building a more comprehensive auth solution for Hanami, so I'm not sure this is worth updating. We'll still use the code from this PR for reference though.

greggilbert commented 7 years ago

@AlfonsoUceda Ah sorry, I got really busy with a bunch of other stuff and didn't get the chance to write more. I'll defer to @cllns' work.

cllns commented 7 years ago

Gonna close this for now then. Thanks for the work @greggilbert, I'll definitely reference the code here in the future