-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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(datepicker): Add optional "today" button #5786
feat(datepicker): Add optional "today" button #5786
Conversation
sampled heavily from quartepie valor-software#5024
sampled heavily from quartepie valor-software#5024
fixed failing test
Hey @daniloff200 , thanks for reviewing. I pulled in latest development and re-ran locally and it seems to be working. Screen shot taken after running the steps:
Is there a better command to verify it is working locally? |
That's strange, I just completely rebuild all the app, and now it works Gonna add some param for that later, and merge that Thanks for your help, @pjlinn !!! |
No worries! Thank you again for the review (and ngx-bootstrap in general) 👍 Good idea about position. |
Add button for user to easily set today's date. Samples heavily from, and finishes closed PR: #5024
Potentially resolves #5680
PR Checklist
Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.