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

Most is done #133

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Most is done #133

wants to merge 2 commits into from

Conversation

abhumbla
Copy link

stuff won't work. but it's close. the libraries need to be installed and idk if everything is in the correct format. additionally, i still need to write a parser function that converts person id to person name in the output string

@abhumbla abhumbla closed this Apr 24, 2015
@abhumbla abhumbla reopened this Apr 24, 2015
@dhe95
Copy link
Contributor

dhe95 commented Apr 24, 2015

iirc you can use the database to translate ids into names with something like

Person.where(:id => user_id)[0]

which will return the Person object, and then you can output the name & email address
the accessable attributes are the column names that can be found in app/models/person.rb

@abhumbla
Copy link
Author

I just used the Person.find_by_id method. I think this code should work. I'll run some more tests tonight, but it looks good to me.

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.

2 participants