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

Require users to login in order to follow friends from other sites. #714

Merged
merged 2 commits into from Nov 16, 2012

Conversation

usmanbashir
Copy link
Member

This implements a fix for Issue 713.

…d out users shouldn't be able to access it without logging in.
…by redirecting and rendering content without terminating the execution of the action as only one of them should be called.
wilkie added a commit that referenced this pull request Nov 16, 2012
…iptions

Require users to login in order to follow friends from other sites using the follow user static page.

Closes #713
@wilkie wilkie merged commit f47bbd9 into hotsh:master Nov 16, 2012
@carols10cents
Copy link
Contributor

Thank you so much @usmanbashir!!!! We'll get you on the about page soon and I'm adding you to the commit access team right now ❤️ ❤️ ❤️

@carols10cents
Copy link
Contributor

Aaand done. Please read this section of the README about what being a committer means: https://github.com/hotsh/rstat.us#becoming-a-committer

Looking forward to more contributions from you!! ❤️

@usmanbashir
Copy link
Member Author

Thanks for accepting my first pull request to an open-source project and adding me to the team. 😃

I look forward to making many more contributions. ❤️

@steveklabnik
Copy link
Contributor

Awesome. :D

@wilkie
Copy link
Contributor

wilkie commented Nov 16, 2012

Welcome to the team. :)

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

4 participants