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

Allow for options on 0.11.2 #77

Closed
wants to merge 11 commits into from
Closed

Conversation

joeytheman
Copy link

Allow for options to be passed in to timeago for 0.11.2. Used code from pull/24

@rmm5t
Copy link
Owner

rmm5t commented May 14, 2012

Thanks for this, but at first glance, it appears to break the test suite. I think it's because the settings argument isn't always optional.

@joeytheman
Copy link
Author

Can you direct me on how to run the test suite so I can take a look at it?

@rmm5t
Copy link
Owner

rmm5t commented May 14, 2012

open test/index.html

@jonathaningram
Copy link

This seems like a good PR, because on the same page, I might have to different instances of timeago I want to handle. E.g.

  1. A grid where I show the last login date of a user. In this case, I want to see "About 21 hours ago" in a column. Note: the A in About is uppercase because it starts a sentence.
  2. On the same page I want to see audit logs of the creation of users. E.g., I want to see "Jon created a new user about 5 minutes ago". Note: the a in about is lowercase because it's in a sentence.

Perhaps you can do this already and I just can't find the docs, but it seems that enabling custom settings per timeago instance would be great.

@joeytheman
Copy link
Author

I fixed the test suite not passing and updated to 11.4. Let me know if you need anything else.

@joeytheman joeytheman closed this Jan 29, 2020
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

3 participants