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

adding item created to the table #34

Merged
merged 1 commit into from
Feb 7, 2014
Merged

Conversation

barretts
Copy link
Contributor

@barretts barretts commented Feb 7, 2014

I wanted to see when the listing was created so I added another column to hold the item.created value seen in NewsItem schema

I wanted to see when the listing was created so I added another column to hold the item.created value seen in NewsItem schema
@megamattron
Copy link
Member

Sweet, add your username to the userlist file too and update the PR and I'll try to merge. More details here: http://pullup.herokuapp.com/signup

barretts added a commit to barretts/pullup that referenced this pull request Feb 7, 2014
Added user name in connection with pull request larvalabs#34
@barretts barretts mentioned this pull request Feb 7, 2014
@barretts
Copy link
Contributor Author

barretts commented Feb 7, 2014

Sorry I think I ended up making another pull request. My first pull request experience!

Sent from Windows Mail

From: Matt Hall
Sent: ‎Friday‎, ‎February‎ ‎7‎, ‎2014 ‎8‎:‎26‎ ‎AM
To: larvalabs/pullup
Cc: Barrett Sonntag

Sweet, add your username to the userlist file too and update the PR and I'll try to merge. More details here: http://pullup.herokuapp.com/signup


Reply to this email directly or view it on GitHub.

@megamattron megamattron merged commit 1dc23a6 into larvalabs:master Feb 7, 2014
@megamattron
Copy link
Member

Ok I merged this, but I switch to showing "timeago" notation which is a little easier to read. We might want to move or deprioritize the placement too, but it's in there for now. Thanks!

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