Skip to content
This repository was archived by the owner on Dec 26, 2019. It is now read-only.

Conversation

jelhan
Copy link
Contributor

@jelhan jelhan commented Aug 15, 2016

Test was broken since it was introduced by #61. Have to call .datepicker('update') to notify about changes.

I'm still not convinced about this test. It seems like testing bootstrap-datepicker and not this addon. But lets get a working testing suite again first.

@soulim
Copy link
Owner

soulim commented Aug 15, 2016

@jelhan you rock! 😄 👍

@soulim soulim merged commit 43683d6 into soulim:master Aug 15, 2016
@octronic
Copy link
Contributor

This test was originally introduced with #35 (which solved #33)

With #63 it was refactored into an integration-test (see: 2c0ca2f)

But this test was removed with 7c53cf0

Long story short: my guess is #61 added this test again by accident.

@soulim
Copy link
Owner

soulim commented Aug 18, 2016

Thanks for the investigation @octronic! 😉👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants