Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

fix(datepicker): date formatting when using angular 1.3 fixes #2659 #2733

Closed
wants to merge 2 commits into from

Conversation

DaveWM
Copy link
Contributor

@DaveWM DaveWM commented Sep 23, 2014

I've put a fix in for this, as suggested by @bcronje. I also had to parse the $viewValue to a date in the $render function.

@davisford
Copy link

Would love to see this merged. I am pinned to an older version of angular until this gets resolved.

@m4r71n
Copy link

m4r71n commented Sep 29, 2014

👍

@piffy1
Copy link

piffy1 commented Oct 10, 2014

Could you let us know when this fix will be released? Thanks.

@homerjam
Copy link

👍

@chrisva
Copy link

chrisva commented Nov 20, 2014

Is this issue fixed and merged?

@e-cloud
Copy link

e-cloud commented Dec 8, 2014

it haven't been merged~! hope soon

@antoinepairet antoinepairet self-assigned this Jan 6, 2015
@antoinepairet
Copy link

LGTM, I just tested it using angular 1.3 and all tests pass. I will discuss this with @pkozlowski-opensource at NG-NL and we will move forward.

@antoinepairet
Copy link

For reference: #3293

@antoinepairet
Copy link

@DaveWM Thanks a lot.
I cherry picked your changes. It landed in this commit: 5f9afe5

see the working plunkr: http://plnkr.co/edit/0EEK4JpvW7GXqbhAgVwJ?p=preview

regrads

@karianna karianna modified the milestones: 0.13.0, Backlog Feb 21, 2015
antoinepairet pushed a commit that referenced this pull request Feb 21, 2015
Closes #3293
Closes #3279
Closes #2440
Closes #2932
Closes #3074
Closes #2943
Closes #2733

Fixes #3047
Fixes #2659
Fixes #2681
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants