Skip to content

Board filter - #1507

Merged
jakobroehrl merged 8 commits into
masterfrom
enh/filter
Feb 6, 2020
Merged

jakobroehrl merged 8 commits into
masterfrom
enh/filter

Conversation

@jakobroehrl

@jakobroehrl jakobroehrl commented Jan 29, 2020 •

Copy link
Copy Markdown
Contributor
  • Target version: master

Todo

  • due filter reset on click again (@change is not reacting)

grafik

Comment thread src/components/board/TagsTabSidebar.vue Outdated
@juliusknorr

Copy link
Copy Markdown
Member

@jakobroehrl I've rebased the branch to current master, please make sure to reset your local branch to the origin one before continuing.

Comment thread src/components/Controls.vue Outdated
Comment thread src/components/Controls.vue Outdated
Comment thread src/store/card.js Outdated
Comment thread src/store/main.js Outdated
@juliusknorr

Copy link
Copy Markdown
Member

@jakobroehrl Pushed some fixes, feel free to take over again now 馃槈

@jakobroehrl jakobroehrl self-assigned this Jan 31, 2020
@jakobroehrl jakobroehrl added this to the 馃殌 Next major 1.0.0 milestone Jan 31, 2020
@jakobroehrl
jakobroehrl force-pushed the enh/filter branch 2 times, most recently from df63fe0 to 0d3299d Compare February 6, 2020 08:04
Comment thread src/store/card.js Outdated
Comment thread src/store/card.js Outdated
@jakobroehrl
jakobroehrl force-pushed the enh/filter branch 2 times, most recently from 1bdacda to 84046d8 Compare February 6, 2020 10:19
@juliusknorr

Copy link
Copy Markdown
Member

Lint is not happy, also as mentioned please rebase instead of creating merge commits, that allows us to maintain clean git log in the master branch.

Comment thread img/filter.svg Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, what is exactly to do?

Comment thread src/components/Controls.vue Outdated
Comment thread src/components/Controls.vue Outdated
Comment thread src/components/Controls.vue Outdated
Comment thread src/store/card.js Outdated
Comment thread src/store/card.js Outdated
Comment thread src/store/card.js Outdated
jakobroehrl and others added 5 commits February 6, 2020 12:35
Signed-off-by: Jakob R枚hrl <[email protected]>
Signed-off-by: Jakob R枚hrl <[email protected]>
Signed-off-by: Jakob R枚hrl <[email protected]>
Signed-off-by: Jakob R枚hrl <[email protected]>
@jakobroehrl
jakobroehrl force-pushed the enh/filter branch 2 times, most recently from 1249bc1 to 1cb7152 Compare February 6, 2020 14:08
Signed-off-by: Jakob R枚hrl <[email protected]>
@juliusknorr

Copy link
Copy Markdown
Member

馃殌

@jakobroehrl
jakobroehrl merged commit 2a94c53 into master Feb 6, 2020
@delete-merged-branch
delete-merged-branch Bot deleted the enh/filter branch February 6, 2020 16:47
@ilippert

Copy link
Copy Markdown

Would it be possible to apply this filter across boards? I.e. at apps/deck/#!/ apply the filter and get the results? That would be very helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants