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.

Sign up for GitHub

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

Jump to bottom

fix:jans-cli Ask on delete of scripts conf. properties #3105

Closed
AbdelwahabAdam opened this issue Nov 29, 2022 · 1 comment
Closed

fix:jans-cli Ask on delete of scripts conf. properties #3105

AbdelwahabAdam opened this issue Nov 29, 2022 · 1 comment
Assignees
@AbdelwahabAdam
Labels
comp-jans-cli-tui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality

Comments

@AbdelwahabAdam
Copy link
Contributor

Description

  • when deleting conf.properties or module properties it happens instantaneous (didn't ask the user to confirm the deletion)
@AbdelwahabAdam AbdelwahabAdam added kind-bug Issue or PR is a bug in existing functionality comp-jans-cli-tui Component affected by issue or PR labels Nov 29, 2022
@AbdelwahabAdam AbdelwahabAdam self-assigned this Nov 29, 2022
AbdelwahabAdam added a commit that referenced this issue Nov 29, 2022
@AbdelwahabAdam
fix:jans-cli Ask on delete of scripts conf. properties (ref: #3105)
3469e03
@AbdelwahabAdam
Copy link
Contributor Author

SOLVED

script_deletewithask.mp4

moabu added a commit that referenced this issue Nov 29, 2022
@moabu @devrimyatar @AbdelwahabAdam
feat: jans cli to jans-cli-tui (#3063)
fc20e28 
* 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
Assignees

@AbdelwahabAdam AbdelwahabAdam

Labels
comp-jans-cli-tui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@AbdelwahabAdam