Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Add Slackbot shortened commands #197

Closed
evan-scott opened this issue Jan 24, 2022 · 2 comments · Fixed by #204
Closed

Add Slackbot shortened commands #197

evan-scott opened this issue Jan 24, 2022 · 2 comments · Fixed by #204
Labels
enhancement New feature or request

Comments

@evan-scott
Copy link

Is your feature request related to a problem? Please describe.
TCEU wants to type 'ar [resource]' rather than 'access request [resource]', partially because it's a lot to type and partially because they're envisioning this in a production incident scenario where every second counts.

Describe the solution you'd like
Allow accessbot to understand both 'access request' and 'ar' as suitable commands, along with any others that can be shortened.

Describe alternatives you've considered
N/A, open to suggestions though.

Additional context
https://app.gong.io/call?id=8746579382246779781&highlights=%5B%7B%22type%22%3A%22SHARE%22%2C%22from%22%3A1057%2C%22to%22%3A1791%7D%5D

IMPORTANT: Remember to select a label and project - I can't :(

@camposer
Copy link
Contributor

camposer commented Jan 26, 2022

Allow the user to configure command aliases via configuration:

COMMAND_ALIASES = {
  "show_resources": "sr",
  "access_resource": "ar",
  "assign_role": None,
  "show_roles": None,
  "approve": None
}

vassalo added a commit that referenced this issue Feb 3, 2022
wallrony added a commit that referenced this issue Feb 3, 2022
Co-authored-by: vassalo <[email protected]>
vassalo added a commit that referenced this issue Feb 3, 2022
@vassalo vassalo added the enhancement New feature or request label Feb 3, 2022
vassalo added a commit that referenced this issue Feb 7, 2022
wallrony added a commit that referenced this issue Feb 8, 2022
camposer added a commit that referenced this issue Feb 10, 2022
@camposer
Copy link
Contributor

camposer commented Feb 10, 2022

We ended up adding the aliases to the COMMANDS_ENABLED variable. I usually go with:

SDM_COMMANDS_ENABLED="access_resource:ar assign_role show_resources:sr show_roles approve:y deny"

So the alias for show available resources is sr

vassalo added a commit that referenced this issue Feb 16, 2022
Co-authored-by: gaabrieljesuss <[email protected]>
Co-authored-by: devlucassantos <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants