bug Success

User tests: Successful: Unsuccessful:

avatar allenzhao
allenzhao
14 Jul 2014

This should solve the bug to display the right month.
@b2z

avatar allenzhao allenzhao - open - 14 Jul 2014
avatar b2z
b2z - comment - 15 Jul 2014

Is it possible to get the month in two digits format ? Not 7 for example, but 07, like 2014-07-15 ?

avatar allenzhao
allenzhao - comment - 15 Jul 2014

Found on stackoverflow that ("0" + (.getMonth() + 1)).slice(-2) might help.

I’m on this now;)

BTW, do we need the date to be two digit?

Fo
-- 
Allen Zhao
Sent with Airmail

On July 15, 2014 at 12:23:58 PM, Dmitry Rekun (notifications@github.com) wrote:

Is it possible to get the month in two digits format ? Not 7 for example, but 07, like 2014-07-15 ?


Reply to this email directly or view it on GitHub.

avatar b2z
b2z - comment - 15 Jul 2014

May be not, I do not insist :)

Best regards,
Dmitry

----- Reply message -----
From: "Zehan Zhao" notifications@github.com
To: "joomla/jissues" jissues@noreply.github.com
Subject: [jissues] Quick fix (#380)
Date: Tue, Jul 15, 2014 07:35
Found on stackoverflow that ("0" + (.getMonth() + 1)).slice(-2) might help.

I’m on this now;)

BTW, do we need the date to be two digit?

Fo

-- 

Allen Zhao

Sent with Airmail

On July 15, 2014 at 12:23:58 PM, Dmitry Rekun (notifications@github.com) wrote:

Is it possible to get the month in two digits format ? Not 7 for example, but 07, like 2014-07-15 ?

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub.

avatar b2z
b2z - comment - 16 Jul 2014

Hmm, date is not two digits. For me it looks a little bit ugly :)

BTW you made two digits on month not date? :)

avatar allenzhao
allenzhao - comment - 17 Jul 2014

Yes, I thought you meant month.
Then I will implement on month too.
You may blame the J!Tracker Application for transmitting this comment.

avatar b2z
b2z - comment - 17 Jul 2014

One more bug in Firefox - time is displayed as NaN-NaN-NaN

Please see this discussion how to fix it.

avatar allenzhao
allenzhao - comment - 19 Jul 2014

Hmm, to avoid this, shall I do it on the server side?
In 4ec3acd, I converted the date to what we need.

avatar allenzhao
allenzhao - comment - 19 Jul 2014

Seems like there's an error for the Travis CI build on the composer...
See here

avatar b2z
b2z - comment - 19 Jul 2014

I see. Not a problem. Sync with the master (some changes were made) and will see how it goes.

avatar infograf768
infograf768 - comment - 19 Jul 2014

Travis has its ups and downs... I had to relaunch it on master multiple times for some PRs.
I have no permission to do it on this repo.

avatar elkuku
elkuku - comment - 19 Jul 2014

Restarted...
Those Travis "failures" are really annoying. We should use our own Pull Tester again :wink:

avatar b2z b2z - close - 19 Jul 2014
avatar b2z b2z - reference | - 19 Jul 14
avatar b2z b2z - merge - 19 Jul 2014
avatar b2z b2z - close - 19 Jul 2014
avatar b2z b2z - change - 19 Jul 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-07-19 17:21:27
avatar b2z
b2z - comment - 19 Jul 2014

:+1:

avatar allenzhao allenzhao - head_ref_deleted - 25 Jul 2014

Add a Comment

Login with GitHub to post a comment