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

fix: Handle valid date provided as string #224 #228

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rahuldimri
Copy link

It is common use-case to provide date as a valid string. Even the backend responses come as a plain string which represents the date.

It is common use-case to provide date as a valid string. Even the backend responses come as a plain string which represents the date.
rahuldimri added a commit to rahuldimri/ngx-pipes that referenced this pull request Jun 14, 2022
@rahuldimri
Copy link
Author

@danrevah Can you please have a look into this.

@rahuldimri
Copy link
Author

@shaungrady @tiger-seo @felladrin @spyr0s Can you please review this request. Its been more than a month.

@felladrin
Copy link
Contributor

Sorry, @rahuldimri, but it seems @danrevah is the only one with write access to this repository. So you'll need to wait for their review.

@rahuldimri
Copy link
Author

@felladrin sure thanks !!

@rahuldimri
Copy link
Author

@danrevah Any idea when we are going to merge this change.

@rahuldimri
Copy link
Author

@danrevah any idea when are you going to merge this request.

@Halynsky
Copy link

@danrevah Ping. Maybe you can give write access to other main contributors?

@brianwarner
Copy link

Hi @danrevah, would it be possible to have a look at this and #229?

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.

5 participants