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

Do not filter statuses with unknown languages #5045

Merged
merged 1 commit into from Sep 22, 2017

Conversation

@akihikodaki
Copy link
Collaborator

commented Sep 22, 2017

No description provided.

@akihikodaki akihikodaki added the bug label Sep 22, 2017
@akihikodaki akihikodaki force-pushed the kagucho:language branch from cc3e65f to a929408 Sep 22, 2017
@akihikodaki akihikodaki changed the title Does not filter statuses with unknown languages Do not filter statuses with unknown languages Sep 22, 2017
end

it 'includes status with unknown language' do
user = Fabricate(:user)

This comment has been minimized.

Copy link
@unarist

unarist Sep 22, 2017

Collaborator

This should have filtered_languages, because language in ... condition will be omitted unless the user has filtered_languages.

@akihikodaki akihikodaki force-pushed the kagucho:language branch from a929408 to d144f6d Sep 22, 2017
@akihikodaki

This comment has been minimized.

Copy link
Collaborator Author

commented Sep 22, 2017

@Gargron Gargron merged commit 0de82dd into tootsuite:master Sep 22, 2017
2 checks passed
2 checks passed
codeclimate All good!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
tateisu pushed a commit to tateisu/mastodon that referenced this pull request Sep 23, 2017
rutan added a commit to rutan/mastodon that referenced this pull request Oct 11, 2017
takayamaki added a commit to takayamaki/mastodon that referenced this pull request Oct 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.