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

fix indexer remove msg with aliases #18885

Merged
merged 19 commits into from Aug 13, 2019

Conversation

@mmaxim
Copy link
Member

commented Aug 12, 2019

  1. Only take out aliases when there is no token to refer to.

mmaxim added some commits Aug 12, 2019

@mmaxim mmaxim marked this pull request as ready for review Aug 12, 2019

@mmaxim mmaxim requested a review from joshblum Aug 12, 2019

go/chat/utils/utils.go Outdated Show resolved Hide resolved

mmaxim added some commits Aug 12, 2019

wip
go/chat/search/storage.go Outdated Show resolved Hide resolved
go/chat/search/storage.go Outdated Show resolved Hide resolved

mmaxim and others added some commits Aug 12, 2019

Update go/chat/search/storage.go
Co-Authored-By: Joshua Blum <joshblum@keyba.se>
@joshblum

This comment has been minimized.

Copy link
Member

commented on go/chat/search/storage.go in a2e0145 Aug 12, 2019

why?

@mmaxim mmaxim merged commit 32fc9b7 into master Aug 13, 2019

0 of 2 checks passed

ci/circleci CircleCI is running your tests
Details
continuous-integration/jenkins/pr-head This commit is being built
Details

@mmaxim mmaxim deleted the mike/HOTPOT-542 branch Aug 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.