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

feat:(core) Introduction of Angular DatePipe #1576

Merged
merged 3 commits into from
Nov 20, 2019

Conversation

JKMarkowski
Copy link
Contributor

@JKMarkowski JKMarkowski commented Nov 12, 2019

Please provide a link to the associated issue.

fixes: #1388

Please provide a brief summary of this pull request.

There is added angular's date pipe, which took responsibility of formatting Date models to string.
It has no compact on default formatting/parsing, default format is the same as before.

Please check whether the PR fulfills the following requirements

Documentation checklist:

@JKMarkowski JKMarkowski changed the title POC: ng pipe datepicker POC: Introduction of Angular DatePipe Nov 12, 2019
@netlify
Copy link

netlify bot commented Nov 12, 2019

Deploy preview for fundamental-ngx ready!

Built with commit ab050d7

https://deploy-preview-1576--fundamental-ngx.netlify.com

@droshev
Copy link
Contributor

droshev commented Nov 17, 2019

@JKMarkowski is this just a POC or a PR for merging?

@JKMarkowski JKMarkowski changed the title POC: Introduction of Angular DatePipe feat: Introduction of Angular DatePipe Nov 18, 2019
@JKMarkowski JKMarkowski changed the title feat: Introduction of Angular DatePipe feat:(core) Introduction of Angular DatePipe Nov 18, 2019
@JKMarkowski
Copy link
Contributor Author

@droshev I think we can treat it like normal PR.

@JKMarkowski JKMarkowski merged commit 8fb91bf into master Nov 20, 2019
@JKMarkowski JKMarkowski deleted the exp/ng-pipe-datepicker branch November 20, 2019 09:12
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.

DatePicker Angular compatibility (critical for usage)
4 participants