Skip to content

Invoke-DbaAdvancedRestore - Fix for proper progress bar display#4454

Merged
potatoqualitee merged 11 commits intodataplat:developmentfrom
olegstrutinskii:development
Nov 9, 2018
Merged

Invoke-DbaAdvancedRestore - Fix for proper progress bar display#4454
potatoqualitee merged 11 commits intodataplat:developmentfrom
olegstrutinskii:development

Conversation

@olegstrutinskii
Copy link
Contributor

@olegstrutinskii olegstrutinskii commented Nov 9, 2018

Type of Change

  • Bug fix (non-breaking change, fixes #)
  • New feature (non-breaking change, adds functionality)
  • Breaking change (effects multiple commands or functionality)
  • Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
  • Adding code coverage to existing functionality
  • Pester test is included
  • Nunit test is included
  • Documentation
  • Build system

Purpose

Somehow progress bar was broken

Approach

Fixed progress ID parenthesis and proper completing
Some 'house cleaning' included too

Commands to test

Restore any database with Restore-DbaDatabase

Screenshots

image

@potatoqualitee
Copy link
Member

potatoqualitee commented Nov 9, 2018

Fantastic!! I've been looking forward to this one for a while, thanks so much @zhunya ! Looking fwd to the review from Stuart or Niph

@olegstrutinskii olegstrutinskii changed the title Fix for proper progress bar display Invoke-DbaAdvancedRestore - Fix for proper progress bar display Nov 9, 2018
@potatoqualitee potatoqualitee merged commit d8d0b9d into dataplat:development Nov 9, 2018
@potatoqualitee
Copy link
Member

Arrgh, so sorry, I thought I was on my PR. I'll do a bunch of testing to ensure this works, the code makes sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants