Skip to content

devise for user authentication and paperclip for avatar#21

Open
ashishrudra wants to merge 3 commits intokeep-delivering:masterfrom
ashishrudra:a/devise
Open

devise for user authentication and paperclip for avatar#21
ashishrudra wants to merge 3 commits intokeep-delivering:masterfrom
ashishrudra:a/devise

Conversation

@ashishrudra
Copy link

sep 12 2015 18 20

@ashishrudra ashishrudra changed the title devise for user authentication devise for user authentication and paperclip for avatar Sep 12, 2015
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're going to have to configure a Heroku-compatible email method.

@stevenweber
Copy link
Contributor

Good start! Looks like you need to work these changes into current master, though.

@ashishrudra
Copy link
Author

it's an alternative of twitter authentication. I think we need to choose one of them.

@stevenweber
Copy link
Contributor

I believe Devise supports omniauth, i.e. Twitter.

@stevenweber
Copy link
Contributor

Talked with @ashishrudra offline about this. I'm not opposed to dropping Twitter support for a more standard login approach, but I see much more work ahead of us with this approach:

  1. Getting email support set up
  2. Getting S3 set up and integrated
  3. Internationalization the generated templates
  4. Updating generated migrations to work with our current structure

Plus I hear we may be able to require emails with our existing omniauth approach, which I think is key to either approach.

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.

2 participants