-
-
Notifications
You must be signed in to change notification settings - Fork 861
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
$te will return false for empty strings #515
Closed
MrWook opened this issue
Feb 5, 2019
· 2 comments
· May be fixed by tt9133github/snyktest#3 or ajesse11x/vuetify#5
Closed
$te will return false for empty strings #515
MrWook opened this issue
Feb 5, 2019
· 2 comments
· May be fixed by tt9133github/snyktest#3 or ajesse11x/vuetify#5
Labels
Comments
kazupon
added
Type: Bug
Bug or Bug fixes
help wanted
Extra attention is needed
good first issue
Good for newcomers
and removed
help wanted
Extra attention is needed
labels
Feb 10, 2019
Hi @kazupon, can i get this one? 😬 |
@kimuraz |
kimuraz
added a commit
to kimuraz/vue-i18n
that referenced
this issue
Mar 2, 2019
kimuraz
added a commit
to kimuraz/vue-i18n
that referenced
this issue
Mar 2, 2019
kimuraz
added a commit
to kimuraz/vue-i18n
that referenced
this issue
Mar 2, 2019
kazupon
pushed a commit
that referenced
this issue
Mar 3, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With the newest version (probably with this commit bed9c39)
$te
will returnfalse
for empty Strings.Before the update it returned
true
for empty Strings.vue & vue-i18n version
ex: 2.5.2, 8.8.8
Reproduction Link
https://codesandbox.io/s/l4mlj428jl
Here is the same repo with the previous version:
https://codesandbox.io/s/ox9vj9x0l9
Steps to reproduce
What is Expected?
What is actually happening?
hi hello world! false
The text was updated successfully, but these errors were encountered: