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.

Sign up for GitHub

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

Jump to bottom

Consolidate import-data commands into one #594

Merged
merged 3 commits into from
Jun 15, 2022
Merged

Consolidate import-data commands into one #594

merged 3 commits into from
Jun 15, 2022

Conversation

dirkmc
Copy link
Contributor

@dirkmc dirkmc commented Jun 14, 2022 •
edited
Loading

Fixes #579

TODO:

  • Test import data for offline v1.2.0 deal by deal uuid
  • Test import data for offline v1.2.0 deal by signed proposal cid
  • Test import data for offline v1.1.0 deal by signed proposal cid

@dirkmc dirkmc marked this pull request as ready for review June 14, 2022 15:43
@dirkmc dirkmc requested a review from nonsense June 14, 2022 15:43
nonsense
nonsense reviewed Jun 15, 2022
View reviewed changes
nonsense
nonsense reviewed Jun 15, 2022
View reviewed changes
dirkmc and others added 2 commits June 15, 2022 13:03
@dirkmc @nonsense
Update cmd/boostd/import_data.go
d871ca4 
Co-authored-by: Anton Evangelatov <[email protected]>
@dirkmc @nonsense
Update cmd/boostd/import_data.go
103ed6b 
Co-authored-by: Anton Evangelatov <[email protected]>
@dirkmc dirkmc merged commit b97aa6d into main Jun 15, 2022
@dirkmc dirkmc deleted the feat/offline-cli-consolidate branch June 15, 2022 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nonsense nonsense nonsense approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consolidate import-data commands
2 participants
@dirkmc @nonsense