Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

reintroduced Installation List with search capability AGPUSH-1406 #577

Merged
merged 1 commit into from
Jun 2, 2015

Conversation

lfryc
Copy link
Contributor

@lfryc lfryc commented Jun 1, 2015

https://issues.jboss.org/browse/AGPUSH-1406

Summary:

  • the device list is conditionally presented to a user in the Variant detail
    • the device list has: search capability
    • ...pagination
    • ability to enable/disable device
  • the client device registration snippet is not shown when there are registered devices (but user can click to reveal it by hitting a link)
  • and vice versa, no device list is shown when there are "No installations yet" and client device registration snippet is presented then

Note:

  • I had to extend REST endpoint by one optional parameter "search" that allows full text search in some comment installation properties

Testing:

  • test that features above works as you would expect

A mockup I used as a base for impl: http://andresgalante.com/ups-console/app-detail-variants.html

@lfryc
Copy link
Contributor Author

lfryc commented Jun 1, 2015

@sebastienblanc would you mind to 👀 this? 🏈

@sebastienblanc
Copy link
Member

👀

@matzew
Copy link
Contributor

matzew commented Jun 2, 2015

+1 tested and works fine

@matzew
Copy link
Contributor

matzew commented Jun 2, 2015

@lfryc @sebastienblanc I think this needs a rebase

@lfryc
Copy link
Contributor Author

lfryc commented Jun 2, 2015

rebasing

@lfryc
Copy link
Contributor Author

lfryc commented Jun 2, 2015

@matzew rebased on latest master

@matzew
Copy link
Contributor

matzew commented Jun 2, 2015

+1

@lfryc
Copy link
Contributor Author

lfryc commented Jun 2, 2015

Once merged, please take a look at #579

@sebastienblanc
Copy link
Member

Working as expected :) , one remark, there is a text + link just before the table This is a list of devices using this variable. To see installation instructions and code snippet click here once on the snippet section there is not really a way to go back to the device list (unless you collapse the variant and open it again), maybe we could add the link to the device list also on the snippet section ?

@matzew
Copy link
Contributor

matzew commented Jun 2, 2015

@sebastienblanc is that really needed ?

@andresgalante any comments?

@lfryc
Copy link
Contributor Author

lfryc commented Jun 2, 2015

if you click on "To see installation instructions and code snippet click here"...
the snippets are expanded, but the device list remains still there, it is now under the snippets
is that what you wanted @sebastienblanc ?

@sebastienblanc
Copy link
Member

Oh Sorry, did not saw it was under the snippets :), then it's all okay :+1

On Tue, Jun 2, 2015 at 1:56 PM, Lukáš Fryč notifications@github.com wrote:

if you click on "To see installation instructions and code snippet click
here"...
the snippets are expanded, but the device list remains still there, it is
now under the snippets
is that what you wanted @sebastienblanc
https://github.com/sebastienblanc ?


Reply to this email directly or view it on GitHub
#577 (comment)
.

@lfryc lfryc merged commit 4e5ef63 into aerogear:master Jun 2, 2015
@lfryc
Copy link
Contributor Author

lfryc commented Jun 2, 2015

Landed 🚀

@lfryc lfryc deleted the installation-list branch June 2, 2015 14:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants