Skip to content

Upd for simple vlontakte auth and registration#4

Open
yhontar wants to merge 20 commits into
EasyDevelopment:yarik_devfrom
yhontar:master
Open

Upd for simple vlontakte auth and registration#4
yhontar wants to merge 20 commits into
EasyDevelopment:yarik_devfrom
yhontar:master

Conversation

@yhontar
Copy link
Copy Markdown

@yhontar yhontar commented Feb 3, 2013

You should upd devise.rb with

config.omniauth :vkontakte, 'APP_ID', 'APP_SECRET'

And run localtunnel -k ~/.ssh/id_rsa.pub 3000 (in development)

@WWTLF
Copy link
Copy Markdown

WWTLF commented Feb 8, 2013

Could you explain me what for "localtunnel -k ~/.ssh/id_rsa.pub 3000 (in development)", please. It is just for my information only.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

are we going to support json? if no, why do we need this format?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

May be in future we will going to develop mobile application:)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think it's just a scaffold-generated code, lets just avoid it and not making "code for future", often it's just a dead code which is hard to maintain and support.

@kiote
Copy link
Copy Markdown
Contributor

kiote commented Feb 8, 2013

@WWTFL you can read about localtunnel here: http://progrium.com/localtunnel/
"The easiest way to share localhost web servers to the rest of the world"

Comment thread Gemfile
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

А зачем этот гем?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

нужно почистить, уж точно не для продакшена :)

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.

4 participants