-
Notifications
You must be signed in to change notification settings - Fork 4
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
Polish the organizations page #29
Comments
Hey, some ideas: Elements:
A project just created/without any content: Comments? @ferblape @nobuti @javivazquez @jorgesancha @bomberstudios |
👍 to get the simplest thing now: RSS, related projects from categories and so on. I'm not sure if I like the bars. I like it because even without friends or updates it might be useful. Should we remove the like (heart icon) and leave only the Follow action (or viceversa)? |
OK to leave aside the like. For the follow, what should we use? https://github.com/tcocca/acts_as_follower? https://github.com/cmer/socialization? |
+1 to acts_as_follower. I wish it works in Rails 4.2 On Sat, Feb 21, 2015 at 8:23 PM, Álvaro Ortiz [email protected]
|
@ferblape, I need your help 🐰 I'm trying to do the follow/unfollow action but can't get the ajax right. I haven't been able to find many updates tutorials, and each one has different code so I'm a little bit lost... You can check the work in this branch: https://github.com/TrackDons/trackdons/tree/29-new-project-page |
Take a look to my latest commit. The secret is that Rails allows you to render |
Ooook! Thanks! So that is the preferred way to do that things? I don't know why thought it would be preferable to do everything inside the coffee file and not using the new js.erb partials ... Its just a matter of taste? |
I guess you can return JSON and manage all the logic in Coffe / JS, but I think it's more complex. Maybe @nobuti has another opinion. |
It's ok for me. |
@jorgesancha said: maybe we could differentiate between most popular (at the top, with logos maybe? and total donations money tracked or similar) and the rest.
The text was updated successfully, but these errors were encountered: