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

index/miners: keep old non-empty location data #142

Merged
merged 2 commits into from Feb 26, 2020
Merged

index/miners: keep old non-empty location data #142

merged 2 commits into from Feb 26, 2020

Conversation

@jsign
Copy link
Contributor

jsign commented Feb 21, 2020

Closes #139

Depends on merging #131

cc @andrewxhill might be interested in this fix when merged and deployed.

@jsign jsign added this to the Sprint 30 milestone Feb 21, 2020
@jsign jsign self-assigned this Feb 21, 2020
@andrewxhill

This comment has been minimized.

Copy link
Member

andrewxhill commented Feb 21, 2020

nice!

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
@jsign jsign force-pushed the jsign/locoverride branch from ecc2d58 to 6f6fcee Feb 25, 2020
@jsign jsign changed the base branch from jsign/fastapi1 to master Feb 25, 2020
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
@jsign jsign marked this pull request as ready for review Feb 25, 2020
@jsign jsign requested review from asutula, andrewxhill and sanderpick Feb 25, 2020
@jsign jsign merged commit 8c20ea2 into master Feb 26, 2020
2 checks passed
2 checks passed
Test
Details
DCO DCO
Details
@jsign jsign deleted the jsign/locoverride branch Feb 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.