Skip to content
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

Added filter by category to article-list #147

Merged

Conversation

wachterjohannes
Copy link
Member

@wachterjohannes wachterjohannes commented Apr 19, 2017

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets none
Related issues/PRs none
License MIT
Documentation PR none

What's in this PR?

This PR add the filter by category button to article-list.

To Do

  • Tests
  • Translation

@wachterjohannes wachterjohannes force-pushed the feature/filter-category branch from bf4a8ef to 3f6dd49 Compare April 19, 2017 09:44
@wachterjohannes wachterjohannes changed the title Added filter by category to article-list [WFP] Added filter by category to article-list Apr 19, 2017
@wachterjohannes wachterjohannes force-pushed the feature/filter-category branch 2 times, most recently from a447dcc to ec239af Compare April 19, 2017 11:06
@wachterjohannes wachterjohannes changed the title [WFP] Added filter by category to article-list [WIP] Added filter by category to article-list Apr 19, 2017
@wachterjohannes wachterjohannes added this to the 0.4.0 milestone Apr 19, 2017
@wachterjohannes wachterjohannes force-pushed the feature/filter-category branch from ec239af to b21e5ed Compare April 19, 2017 12:55
@wachterjohannes wachterjohannes force-pushed the feature/filter-category branch from b21e5ed to f9811fa Compare April 19, 2017 13:23
@wachterjohannes wachterjohannes changed the title [WIP] Added filter by category to article-list Added filter by category to article-list Apr 19, 2017
Copy link
Member

@alexander-schranz alexander-schranz left a comment

Choose a reason for hiding this comment

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

else ACK

* @return {String}
*/
getContactFilterTitle: function(filterKey, contact) {
getFilterTitle: function(filterKey, contact, category) {
var title = '';

switch(filterKey) {
case 'all':
title = this.translations.filterAll;
break;
case 'filterBy':
Copy link
Member

Choose a reason for hiding this comment

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

filterByAuthor?

@wachterjohannes wachterjohannes force-pushed the feature/filter-category branch from 8fa363c to 49c6bbf Compare April 20, 2017 09:47
@alexander-schranz alexander-schranz merged commit bcfce44 into sulu:develop Apr 20, 2017
@wachterjohannes wachterjohannes deleted the feature/filter-category branch April 20, 2017 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants