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

Don't clear the _currentIsoFile if another iso was selected #443

Closed
wants to merge 1 commit into from

Conversation

bcl
Copy link
Contributor

@bcl bcl commented Nov 7, 2015

It was clearing the selection even if another iso was selected. This
resulted in the summary hub displaying 'Error setting up ISO file', but
still allowing installation to continue. It should only do that if
something other than an iso is selected.

It was clearing the selection even if another iso was selected.  This
resulted in the summary hub displaying 'Error setting up ISO file', but
still allowing installation to continue. It should only do that if
something other than an iso is selected.
@bcl bcl added the master Please, use the `f39` label instead. label Nov 7, 2015
@jkonecny12
Copy link
Member

Looks good to me.

@jkonecny12 jkonecny12 added the ACK label Nov 9, 2015
@bcl
Copy link
Contributor Author

bcl commented Nov 10, 2015

Pushed

@bcl bcl closed this Nov 10, 2015
@bcl bcl deleted the f23-repo-hd branch November 10, 2015 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master Please, use the `f39` label instead.
2 participants