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

Enable to download AWS volumes snapshot summary #1457

Merged

Conversation

hstastna
Copy link

@hstastna hstastna commented May 29, 2017

fixing https://bugzilla.redhat.com/show_bug.cgi?id=1439629

Enable to download AWS volumes snapshot summary in PDF format
when reaching Storage > Block Storage > Volume Snapshots page
and clicking on one of the snapshots to get into summary page.

snapshot

fixing https://bugzilla.redhat.com/show_bug.cgi?id=1439629

Enable to download AWS volumes snapshot summary in PDF format
when reaching Storage > Block Storage > Volume Snapshots page
and clicking on one of the snapshots to get into summary page.
@hstastna
Copy link
Author

@miq-bot add_label bug

@miq-bot miq-bot added the bug label May 29, 2017
@miq-bot
Copy link
Member

miq-bot commented May 29, 2017

Checked commit hstastna@54ea9d4 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@mzazrivec mzazrivec self-assigned this May 30, 2017
@mzazrivec mzazrivec added this to the Sprint 62 Ending Jun 5, 2017 milestone May 30, 2017
@mzazrivec mzazrivec merged commit 817d2dd into ManageIQ:master May 30, 2017
@psachin
Copy link

psachin commented May 31, 2017

@hstastna Do we maintain any test-cases for this kind of feature? That should avoid future bugs. I guess we also have similar issue in generating PDF report for network_port

@hstastna
Copy link
Author

hstastna commented May 31, 2017

@psachin I am not sure if we maintain any test-cases for this kind of feature. And could you please specify the issue for network_port? Because if I reach Networks -> Network Ports and choose one of them, downloading PDF summary works fine.

@psachin
Copy link

psachin commented Jun 1, 2017

@hstastna Ah! I don't see any problem downloading PDF summary for network port. My bad. Thanks for the fix.

simaishi pushed a commit that referenced this pull request Jun 9, 2017
@simaishi
Copy link
Contributor

simaishi commented Jun 9, 2017

Fine backport details:

$ git log -1
commit 6fb82096c4e70590126f316eb117d39a7955bf58
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Tue May 30 10:18:31 2017 +0200

    Merge pull request #1457 from hstastna/AWS_volumes_snapshot_summary_PDF
    
    Enable to download AWS volumes snapshot summary
    (cherry picked from commit 817d2dd62b561e97efefca864aac9573ab2f2caf)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1460385

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.

5 participants