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

Fix volume attach/detach flash messages #745

Merged
merged 1 commit into from
Mar 23, 2017

Conversation

gberginc
Copy link
Contributor

Flash message that is displayed right after a volume is attached/detached, doe not properly substitute variables because the volume name variable was incorrect (name instead of volume_name).

This patch updates the flash messagae substitutions.

Before the change:

Attaching Cloud Volume "%{volume_name}" to %{vm_name} finished
Detaching Cloud Volume "%{volume_name}" from %{vm_name} finished

After this change:

Attaching Cloud Volume "vol-aws1" to miq-t2.micro finished
Detaching Cloud Volume "vol-aws1" from miq-t2.micro finished

Flash message that is displayed right after a volume is
attached/detached, doe not properly substitute variables because the
volume name variable was incorrect (`name` instead of `volume_name`).

This patch updates the flash messagae substitutions.

Before the change:

```
Attaching Cloud Volume "%{volume_name}" to %{vm_name} finished
Detaching Cloud Volume "%{volume_name}" from %{vm_name} finished
```

After this change:

```
Attaching Cloud Volume "vol-aws1" to miq-t2.micro finished
Detaching Cloud Volume "vol-aws1" from miq-t2.micro finished
```
@gberginc
Copy link
Contributor Author

@miq-bot add_label bug,storage

@miq-bot
Copy link
Member

miq-bot commented Mar 20, 2017

Checked commit gberginc@1f9cf59 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 👍

@gberginc
Copy link
Contributor Author

@AparnaKarve can you have a look at this one? If not, please assign to someone who can.

@mzazrivec mzazrivec self-assigned this Mar 23, 2017
@mzazrivec mzazrivec added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 23, 2017
@mzazrivec mzazrivec merged commit 207420d into ManageIQ:master Mar 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants