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

(S) Wireframe the browse / search / list groups page #2

Closed
sfinn85 opened this issue Dec 23, 2019 · 11 comments
Closed

(S) Wireframe the browse / search / list groups page #2

sfinn85 opened this issue Dec 23, 2019 · 11 comments
Assignees
Labels
S
Projects

Comments

@sfinn85
Copy link

@sfinn85 sfinn85 commented Dec 23, 2019

No description provided.

@puiterwijk puiterwijk transferred this issue from fedora-infra/freeipa-fas Jan 6, 2020
@sfinn85

This comment has been minimized.

Copy link
Author

@sfinn85 sfinn85 commented Jan 7, 2020

Acceptance Criteria:

Visual wire frame of new user form
Fields ( TBC )
Working doc

DOD:
All above completed

@ryanlerch ryanlerch added this to To do in AAA Jan 7, 2020
@sfinn85 sfinn85 changed the title Wireframe the browse / search / list groups page (S) Wireframe the browse / search / list groups page Jan 7, 2020
@ryanlerch ryanlerch moved this from To do to In progress in AAA Jan 8, 2020
@ryanlerch

This comment has been minimized.

Copy link
Contributor

@ryanlerch ryanlerch commented Jan 8, 2020

first pass at the browse / list / search pages for users and groups

userlist
grouplist

@ryanlerch ryanlerch moved this from In progress to Ready for review in AAA Jan 8, 2020
@StephenCoady

This comment has been minimized.

Copy link
Contributor

@StephenCoady StephenCoady commented Jan 8, 2020

Looks good to me.

@relrod relrod added the S label Jan 8, 2020
@ryanlerch ryanlerch self-assigned this Jan 9, 2020
@sfinn85

This comment has been minimized.

Copy link
Author

@sfinn85 sfinn85 commented Jan 14, 2020

Yep looks good to me.

Few points

  • The list is quite long on one page, is this the norm for these accounts, or do we just have a search bar instead of listing all the groups/ users?
  • If we need to list could be reduce the number shown on each page and have them in alphabetical order?
  • Has anything changed in design from current design that we would need to review from dev perspective?

Thoughts..

@ryanlerch

This comment has been minimized.

Copy link
Contributor

@ryanlerch ryanlerch commented Jan 15, 2020

Therotically, we would want this paginated, but not sure if the ipa backend makes this possible or easy, so didnt put it in the wireframe for now.

@sfinn85

This comment has been minimized.

Copy link
Author

@sfinn85 sfinn85 commented Jan 15, 2020

Cool, thanks @ryanlerch .

@relrod what are your thoughts here? If we can improve UX as we build AAA excellent but only if we have the capacity to do so.

@sfinn85

This comment has been minimized.

Copy link
Author

@sfinn85 sfinn85 commented Jan 20, 2020

Possible solutions from Rick - to discuss. ( UI to be tweaked)

@sfinn85 sfinn85 moved this from Ready for review to To do in AAA Jan 20, 2020
@ryanlerch ryanlerch moved this from To do to In progress in AAA Jan 28, 2020
@sfinn85 sfinn85 moved this from In progress to Blocked in AAA Feb 3, 2020
@relrod

This comment has been minimized.

Copy link
Member

@relrod relrod commented Feb 3, 2020

@ryanlerch Would paginating client-side instead work?

@ryanlerch

This comment has been minimized.

Copy link
Contributor

@ryanlerch ryanlerch commented Feb 3, 2020

@relrod works for me, but only if we are talking loading clientside with ajax. otherwise better to just show all the entries here. It is kind of useless to load all the items in the html, then artificially paginate clientside imho.

@relrod

This comment has been minimized.

Copy link
Member

@relrod relrod commented Feb 3, 2020

@ryanlerch fair. I don't know if the IPA API handles pagination (I can look in a bit). I figured just not dumping everything on the page could look nice from a UX perspective, even if we pull all the results at once. But I'm fine with whatever :)

@sfinn85

This comment has been minimized.

Copy link
Author

@sfinn85 sfinn85 commented Feb 4, 2020

leave wireframe as is, lots of work to change, search functionality available to refine user view

@sfinn85 sfinn85 moved this from Blocked to Done within Sprint in AAA Feb 4, 2020
@sfinn85 sfinn85 closed this Feb 4, 2020
@sfinn85 sfinn85 moved this from Done within Sprint to Overall Done in AAA Feb 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
AAA
  
Overall Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.