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

Incorrect time on dynamic content #137

Open
akalinovskiy opened this issue Jun 3, 2015 · 9 comments
Open

Incorrect time on dynamic content #137

akalinovskiy opened this issue Jun 3, 2015 · 9 comments
Assignees

Comments

@akalinovskiy
Copy link

If we use serverTime option newly created items have incorrect time.
Example - http://jsfiddle.net/2mh8nzq9/

@akalinovskiy
Copy link
Author

I think the problem lies in this line. I suppose 'localDate' variable should be initialized at module definition time.

My ugly hot fix that seems to be working

@alaz
Copy link

alaz commented Jul 10, 2015

@urish ping. Any update on this? I have this bug in production, so it would be great to know should we expect a release. Otherwise we would fork and use the patched library.

@urish
Copy link
Owner

urish commented Jul 10, 2015

@alaz Please see my last comment on #139. Feel free to send an updated PR, otherwise it will have to wait for me to find some free time to look into it...

@alaz
Copy link

alaz commented Jul 10, 2015

Sorry, I missed your that comment somehow

@urish
Copy link
Owner

urish commented Jul 10, 2015

No worries :)

On Fri, Jul 10, 2015 at 1:27 PM, Alexander Azarov [email protected]
wrote:

Sorry, I missed your that comment somehow


Reply to this email directly or view it on GitHub
#137 (comment)
.

@urish urish added this to the 1.0.0 milestone Sep 15, 2015
@urish urish self-assigned this Sep 15, 2015
@pmoelgaard
Copy link

maybe this is the error, however generally I haven't been able to get it to work correctly with amParse and display anything correctly, messes up the years, and doesn't follow the patterns of momentjs, but those of the native DateJS... sorry to see such effort wasted, but have to roll my own...

@urish
Copy link
Owner

urish commented Oct 23, 2015

Hi Peter, can you please elaborate?

@pmoelgaard
Copy link

OK, let me get together a sample of the problem and post it... thanks for the quick reply

@urish
Copy link
Owner

urish commented Oct 23, 2015

Sure, thanks!

@urish urish removed this from the 1.0.0 milestone Oct 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants