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

[WIP] Support Pod to PV relationship #1774

Closed

Conversation

nimrodshn
Copy link
Contributor

Added UI support for the Pod to PV relationship.
Print Screen:
screenshot from 2017-07-27 17-27-53

Related to: ManageIQ/manageiq#15023
cc: @simon3z @himdel @zakiva

@miq-bot miq-bot added the wip label Jul 27, 2017
@nimrodshn
Copy link
Contributor Author

@himdel for some reason the links created using textual_link at the textual_link_pod_to_pv method isn't working properly: on my test setup it keeps returning an empty list even though the back-end shows a list > 0 (as shown in the icon above!) can you share some thoughts about this?
screenshot from 2017-07-27 17-24-06

@miq-bot
Copy link
Member

miq-bot commented Jul 27, 2017

Checked commits nimrodshn/manageiq-ui-classic@3a2fe21~...f543e1c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. ⭐

@himdel
Copy link
Contributor

himdel commented Aug 7, 2017

@nimrodshn I think the link is fine, it's just the GTL on that page isn't working...

(you should see a http request for /(controller)/report_data once that page loads .. maybe the report_data method is not ready for this particular list. - but feel free to create an issue, just add the details of that report_data request in there please :).)

@nimrodshn
Copy link
Contributor Author

@himdel will do that. Thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants