-
Notifications
You must be signed in to change notification settings - Fork 41
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
Feature Request: Package as a Rails engine #116
Comments
Hi @mhoad , Happy to read you find this demo helpful. We've had some discussion recently around how to provide some of this out of the box. |
Haven't thought about a rails engine until now, so we should also consider that as a possibility... Thanks for sharing your ideas. |
Glad to hear from you. I think what you’ve done is a huge contribution to
the overall rails community. I’d love to see it have the opportunity to be
more widely shared. Merry Christmas in the meantime
…On Tue, 24 Dec 2019 at 14:41, Gonzalo Rodriguez ***@***.***> wrote:
Haven't thought about a rails engine until now, so we should also consider
that as a possibility...
Thanks for sharing your ideas.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#116?email_source=notifications&email_token=AABAPDPTONMTEEE5SITY3I3Q2INRJA5CNFSM4J5KIUMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHTJNMY#issuecomment-568759987>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABAPDIFQZ75ET6UH45JVCTQ2INRJANCNFSM4J5KIUMA>
.
|
That's great to hear!
Feel free to join https://gitter.im/cedarcode/webauthn-ruby :-) |
This demo is extremely helpful and I am incredible grateful for it. I was wondering if you would be willing to perhaps look at doing this as a Rails engine that can just be dropped into any kind of Rails project much in the same way that Devise works.
It seems like there is still quite a bit of work between say adding
gem "webauthn"
to the Gemfile and a functional Rails implementation. Given that the work already exists as a part of this repo I wanted to see if this made sense for your plans as I think it would be hugely helpful to the community in general.The text was updated successfully, but these errors were encountered: