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

Add Backup download #397

Merged
merged 8 commits into from Mar 5, 2019
Merged

Add Backup download #397

merged 8 commits into from Mar 5, 2019

Conversation

@johnsudaar
Copy link
Member

@johnsudaar johnsudaar commented Feb 28, 2019

No description provided.

@johnsudaar
Copy link
Member Author

@johnsudaar johnsudaar commented Mar 4, 2019

All the code is there, I'm just waiting on Scalingo/go-scalingo#76 to be merged.

@johnsudaar johnsudaar requested a review from EtienneM Mar 4, 2019
@johnsudaar johnsudaar changed the title Update to latest go-scalingo version Add Backup download Mar 4, 2019
Copy link
Member

@EtienneM EtienneM left a comment

I really like how well commented is the backup download method :)

cmd/backups.go Outdated Show resolved Hide resolved
cmd/backups.go Outdated Show resolved Hide resolved
cmd/backups.go Show resolved Hide resolved
cmd/flags.go Outdated Show resolved Hide resolved
config/auth/config.go Outdated Show resolved Hide resolved
db/backup_download.go Show resolved Hide resolved
db/backup_download.go Outdated Show resolved Hide resolved
db/backups.go Outdated Show resolved Hide resolved
@johnsudaar johnsudaar requested a review from EtienneM Mar 4, 2019
Gopkg.toml Outdated Show resolved Hide resolved
cmd/backups.go Outdated Show resolved Hide resolved
@johnsudaar johnsudaar requested a review from EtienneM Mar 5, 2019
@johnsudaar johnsudaar requested a review from Soulou Mar 5, 2019
@Soulou Soulou merged commit 59681de into master Mar 5, 2019
@EtienneM EtienneM deleted the fix/add_db branch May 17, 2021
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.

None yet

3 participants