-
Notifications
You must be signed in to change notification settings - Fork 100
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
Colw/graphql validators #2926
Colw/graphql validators #2926
Conversation
delegations update Page Validators update Page validator update Filters test update table validator update
Codecov Report
@@ Coverage Diff @@
## develop #2926 +/- ##
===========================================
- Coverage 97.32% 97.09% -0.24%
===========================================
Files 126 126
Lines 2247 2240 -7
Branches 396 391 -5
===========================================
- Hits 2187 2175 -12
- Misses 54 59 +5
Partials 6 6
|
Codecov Report
@@ Coverage Diff @@
## develop #2926 +/- ##
===========================================
- Coverage 97.32% 97.22% -0.11%
===========================================
Files 126 126
Lines 2247 2236 -11
Branches 384 391 +7
===========================================
- Hits 2187 2174 -13
- Misses 54 56 +2
Partials 6 6
|
src/components/common/TmPage.vue
Outdated
@@ -77,7 +77,7 @@ export default { | |||
default: false | |||
}, | |||
error: { | |||
type: Error, | |||
type: [Error, Boolean], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have a rule about not allowing multiple types per property
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call. TMPage
already treated it like a Boolean, so I've removed the Error
type. Other values will be coerced to Boolean.
@@ -503,17 +472,17 @@ span { | |||
border-radius: 0.25rem; | |||
} | |||
|
|||
.validator-status.red { | |||
.validator-status.jailed { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice semantic classes
Closes #ISSUE
Description:
Thank you! 🚀
For contributor:
yarn changelog
for a guided process.Files changed
in the github PR explorerFor reviewer: