-
Notifications
You must be signed in to change notification settings - Fork 7
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
WIP: Add user profile #47
base: master
Are you sure you want to change the base?
Conversation
Benjamin Fleischer Rich Blumer
If user is registered, show profile else show 'no such user' page
I checked out this branch to give it a try, and got a parse error on the regex literal:
It just needed the Incidentally, what's the reason behind stripping characters out of the profile name? |
@joonty just thinking about cleaning params |
Ok, wouldn't ActiveRecord handle it though? In the Sorry if I'm missing the point, which is very possible! |
On Fri, Jun 28, 2013 at 8:54 AM, Jonathan Cairns
No, I just didn't look into it. Will remove in the next iteration. |
As @bf4 and @rich-ware :
To surface user profile behind github authentication per avdi/ppwm#8
Thoughts of the feature 'finding your pair'
Finding Your Pair
User profile page
Registered User features
User profile information