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(jans-cli-tui): getting window too small error in edit user #3041

Closed
manojs1978 opened this issue Nov 21, 2022 · 4 comments
Closed

fix(jans-cli-tui): getting window too small error in edit user #3041

manojs1978 opened this issue Nov 21, 2022 · 4 comments
Assignees
Labels
comp-jans-tui kind-bug Issue or PR is a bug in existing functionality

Comments

@manojs1978
Copy link
Contributor

Describe the bug
getting window too small error in edit user

To Reproduce
Steps to reproduce the behavior:

  1. launch jans -tui
  2. goto edit user
  3. edit existing user
  4. See error

Expected behavior
should not get any error

Screenshots
image

Desktop (please complete the following information):

  • OS: ubuntu
  • Browser :
  • Version 20.04
  • DB openDJ

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@manojs1978 manojs1978 added kind-bug Issue or PR is a bug in existing functionality comp-jans-tui labels Nov 21, 2022
@devrimyatar
Copy link
Contributor

I can't replicate, even with very small window, it can display edit user dialog

image

@manojs1978
Copy link
Contributor Author

hello @devrimyatar,
please select "jans auth test user" from list
image

@manojs1978
Copy link
Contributor Author

when we add multiple claims while add/edit user this issue is reproducible.

image

@AbdelwahabAdam
Copy link
Contributor

Solved

Users_error_fix.mp4

moabu added a commit that referenced this issue Nov 29, 2022
* refactor: remove jans-cli

* fix: jans-cli no need to get properties each time

* feat: jans-linux-setup tui installer

* fix: jans-linux-setup clean old cli user data

* fix: jans-linux-setup missing cli libraries

* fix: jans-cli code clean OrderedDict

* fix: jans-cli runtime plugins

* fix: jans-cli don't decode if access_token_enc is empty

* fix: jans-cli edit property (ref: #3058)

* fix: jans-cli screen hangs on scim save (ref: #3061)

* fix: jans-cli multiple device auth message on save creds (ref: #3059)

* fix:jans-cli remove unnecessary logs

* fix:jans-cli reviced all array type in properties (ref: #3062)

* fix:jans-cli fix move backword in auth server column (ref: #\3073)

* fix: jans-cli delete if jca_client_secret manually added to config.ini

* fix: jans-cli typo

* fix: jans-cli config-api move to asyncio

* fix: jans-cli save admin-ui roles

* feat: jans-cli display progressing text on status bar

* feat: jans-cli progressing texts for user plugin

* fix: jans-cli decode encrypted data if salt changed

* fix: jans-cli check existence of data when logout (ref: #3050)

* fix:jans-cli getting window too small error in edit user (ref: #3041)

* fix:jans-cli remove un-used scrollbar (ref: #3070 #3041)

* feat: jans-cli add User-inum to request headers

* fix: jans-cli check existence of inum in user info

* fix: jans-cli DocString (ref: #2572)

* fix: jans-cli DocString (ref: #2572)

* fix: jans-cli DocString and removing unused imports (ref: #2572)

* fix:jans-cli Ask on delete of scripts conf. properties (ref: #3105)

* fix: jans-cli Error on deleting Claims in scopes(ref:  #3106)

* fix: jans-cli Error on deleting without verification of deletion (ref: #3107)

Co-authored-by: Mustafa Baser <mbaser@mail.com>
Co-authored-by: AbdelwahabAdam <abdelwahabosama.1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-tui kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

No branches or pull requests

4 participants