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

add uppercase shortcut for navigation #215

Closed
wants to merge 14 commits into from
Closed

Conversation

outscale-mal
Copy link
Contributor

@outscale-mal outscale-mal commented Aug 21, 2023

🚀 Added / Fixed :

-> Shortcuts to navigate into :

  • Create with C
  • Volumes with V
  • Images with M
  • Instances with I
  • Security with S
  • Nets with T
  • Keypairs with K

-> Fix VPC bug when CIDR is missing by adding error handling

-> Code scanning errors :

  • Unused imports & local variables
  • First parameter of a method is not named 'self'
  • Cyclic imports

-> Refactoring

🚩 Problems :

  • The letter N can't be used as a shortcut, maybe used somewhere already

@outscale-mal
Copy link
Contributor Author

Suggestion: I can reorganize the code for shortcuts to put them in a separate file and make the code less heavy in this file :)

osc_tui/mainForm.py Fixed Show fixed Hide fixed
osc_tui/mainForm.py Fixed Show fixed Hide fixed
osc_tui/mainForm.py Fixed Show fixed Hide fixed
osc_tui/mainForm.py Fixed Show fixed Hide fixed
osc_tui/createVpcs.py Fixed Show fixed Hide fixed
osc_tui/createVpcs.py Fixed Show fixed Hide fixed
osc_tui/createVpcs.py Fixed Show fixed Hide fixed
osc_tui/createVpcs.py Fixed Show fixed Hide fixed
@outscale-mal outscale-mal deleted the add-shortcuts branch September 14, 2023 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant