Conversation
guergana
commented
Dec 22, 2020
- quiet button will be added when new version of wikit is released
- added removeCondition action and mutation
- added tests
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.
Thank you for taking this on! We should extract the component, but that should be relatively
@@ -19,12 +19,19 @@ | |||
:placeholder="$i18n('query-builder-input-value-placeholder')" | |||
:disabled="isTextInputDisabled()" | |||
/> | |||
<Button |
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.
This should probably be its own simple subcomponent as well, similar to ValueTypeDropdown and PropertyLookup. The idea is to try to maintain "one level of abstraction" within the template as well.
But that could also be done in a follow-up pull request
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.
Let's do it in a followup. :)
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.
👍
tests/unit/store/mutations.spec.ts
Outdated
conditionId: '0.123', | ||
}; | ||
const state: RootState = { | ||
conditionRows: [ { |
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.
You can replace this first object with the keptRow
variable from above :)
+ quiet button will be added when new version of wikit is released + added removeCondition action and mutation + added tests
cc18799
to
5b9759c
Compare