Skip to content
This repository has been archived by the owner on Jul 26, 2022. It is now read-only.

Set the default of value to empty string #41

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

Conversation

schwarmco
Copy link

This will fix a bug, i've came across, when giving an undefined value to v-model.

See: https://codepen.io/anon/pen/vPgeRM

@coveralls
Copy link

Pull Request Test Coverage Report for Build 54

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.02%) to 70.408%

Totals Coverage Status
Change from base Build 38: 1.02%
Covered Lines: 48
Relevant Lines: 68

💛 - Coveralls

@mattzollinhofer
Copy link

I'm just plowing through the obvious problems right now to get those fixed/merged. I'm not sure I totally understand the problem here. Can you give me a little more information about the issue?

I've recently merged in some code to a new version of this project. Can you give that a shot and let me know if you're still having trouble?

You can find the new version on NPM here: https://www.npmjs.com/package/vue-typeahead-bootstrap or github here: https://github.com/mattzollinhofer/vue-typeahead-bootstrap.

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