Skip to content
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/13 user sign up #45

Closed
wants to merge 7 commits into from

Conversation

doitian
Copy link
Member

@doitian doitian commented Dec 25, 2012

#13 注册 no more, no less

@doitian
Copy link
Member Author

doitian commented Dec 25, 2012

jruby 又抽风了

@doitian
Copy link
Member Author

doitian commented Dec 25, 2012

没加 form builder, 错误暂时全放 flash 里

@saberma
Copy link
Member

saberma commented Dec 25, 2012

还是用 devise 吧,我们还需要「记住我们」、「邮件确认」等功能,用 devise 很配置一下就行了。

@doitian
Copy link
Member Author

doitian commented Dec 25, 2012

@saberma 嗯,我把 jruby 重新 pr

@@ -0,0 +1,199 @@
zh-CN:
Copy link
Member

Choose a reason for hiding this comment

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

@doitian 这些等需要用上的时候再加上吧,我们尽量保持 PL 最小化。

@saberma
Copy link
Member

saberma commented Dec 27, 2012

已合并 #67

@saberma saberma closed this Dec 27, 2012
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.

None yet

3 participants