-
Notifications
You must be signed in to change notification settings - Fork 10
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
Restore the people list #60
Comments
I'd like to work on this one. here's the commit which removed the people index for reference. Do you want me to more or less undo this commit, minus the filter code, plus the lazy unicorns breakdown? Would you like me to adjust the formatting in some way? Simple list vs. alumni-style grid, decorated or not? Is respond_to js still needed? I'm not clear on whether that was part of the filter. |
It's all yours
I wouldn't. Let's start fresh and just use the old code as a guide.
Start off with something very simple: Gravatar and name. Might make sense to do a few mockups if that's something you are comfortable with.
That was for the filter which we don't need to support in the new version. Good luck ✨ |
Putting the index back in was easy, now I'll definitely want to do mockups. Have we got a set of test data, or a list of people available from an API? Otherwise I'll need to set up factory script a la puzzlenode, or a scraper, or (ugh) data entry. Real data is the best data, as with the Nerian Test ™️ |
Or I could write a scraper which slurps alumni github account names and fetches their email addresses using github's user API so I can do the mockup with their gravatars. 🐑 Hack day is starting early over here. |
re: scraper - I actually did that, and loaded my dev db from a CSV file. As badass as the default unicorn picture is, I wanted to see more than one gravatar on the page :) I tried a basic list view and found it didn't look too great. As this whole thing is provisional, what if I borrow the 3 column style used on the current alumni list? |
Hard to say. I'd have to see the design. |
I'll get you a screenshot when this looks less like I've just pasted styles from one site into another :) |
@jordanbyron If you have time to discuss the above, I think I can wrap this up pretty quickly. |
This lists should be visible to logged in users only.
Split users into two sections:
Top section will list users with profiles (Sorted by name)
Below that section, a header "Lazy Unicorns" will list users without profiles
Paginated. Search isn't required as #12 will take care of that.
The text was updated successfully, but these errors were encountered: