Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Migrate listing hashes in index to new format #1590

Merged
merged 2 commits into from
Jun 6, 2019
Merged

Conversation

cpacia
Copy link
Member

@cpacia cpacia commented May 20, 2019

Migration029 updates the hash of each listing in the listing index to use
the new hash format.

Fixes #1568

Migration029 updates the hash of each listing in the listing index to use
the new hash format.

Fixes #1568
@cpacia cpacia force-pushed the listingmigration branch from f45b7ed to 2a4841f Compare May 20, 2019 15:13
@coveralls
Copy link

coveralls commented May 20, 2019

Coverage Status

Coverage increased (+0.05%) to 35.63% when pulling b271fa6 on listingmigration into cacb790 on master.

Copy link
Member

@placer14 placer14 left a comment

Choose a reason for hiding this comment

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

I have a couple of suggestions and questions for this PR.

repo/migrations/Migration029.go Show resolved Hide resolved
repo/migrations/Migration029.go Outdated Show resolved Hide resolved
repo/migrations/Migration029_test.go Outdated Show resolved Hide resolved
repo/migrations/Migration029_test.go Show resolved Hide resolved
Copy link
Member

@placer14 placer14 left a comment

Choose a reason for hiding this comment

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

@cpacia Just the correction on the variables mentioned above, please.

repo/migrations/Migration029_test.go Outdated Show resolved Hide resolved
@cpacia cpacia force-pushed the listingmigration branch from 91620f0 to b271fa6 Compare May 29, 2019 15:04
@cpacia cpacia merged commit b0bc590 into master Jun 6, 2019
@cpacia cpacia deleted the listingmigration branch July 15, 2019 11:29
@placer14 placer14 changed the title Create migration029 Migrate listing hashes in index to new format Jul 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hash Changed On Unchanged Listing
3 participants