Skip to content
This repository has been archived by the owner on Jul 9, 2021. It is now read-only.

humanize the account creation date #166

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

Conversation

Tanish2002
Copy link
Contributor

so it is easier to identify when the account was created..

@eepykate
Copy link
Contributor

So this will change

Account Creation Date
2017-11-09 12:45

to

Account Creation Date
9th of november, 2017 at 12:45pm

?

I heavily disagree with this change.

@eepykate
Copy link
Contributor

eepykate commented Feb 12, 2021

Oh, nevermind.

it will do

Account Creation Date
2021-02-12 11:22 (28 seconds ago)
Join Date
2021-02-12 11:23 (now)

it seems?

in my defence the email link i opened only showed the most recent commit's diff and i thought it was for all commits...

@Tanish2002
Copy link
Contributor Author

Yes it wil do ( x secs/mins/hours/days/years ago), just like in !info, so it's easier to identify when the account was created. I think this would be especially helpful during the ice invasion

Also it doesn't humanize join date as it will always be (now)

@eepykate eepykate requested a review from mlvzk February 14, 2021 03:12
@Tanish2002
Copy link
Contributor Author

tf even happened?🥲

@eepykate
Copy link
Contributor

ladle made changes deliberately to break your PRs so he doesn't have to merge

amirite @davidv171

@Tanish2002
Copy link
Contributor Author

@6gk no one can stop me 😈

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants