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

add required prop #48

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

add required prop #48

wants to merge 1 commit into from

Conversation

mcshaz
Copy link

@mcshaz mcshaz commented Apr 24, 2019

add the html-5 required attribute to the input element if prop value is true

@coveralls
Copy link

Pull Request Test Coverage Report for Build 63

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 69.388%

Totals Coverage Status
Change from base Build 38: 0.0%
Covered Lines: 47
Relevant Lines: 68

💛 - Coveralls

@mattzollinhofer
Copy link

I'm not sure this PR is fully fleshed out or maybe I'm missing something. If this is still an issue for you could you clarify the use case a bit more? I'm looking to pick this project back up. Unfortunately, I've had to fork it over to here: https://github.com/mattzollinhofer/vue-typeahead-bootstrap/ but I'll try to keep an eye on this repo as well because I know a lot of folks started here.

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