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

Feature Request - Cron schedule - Allow command argument entry #867

Open
veteranmina opened this issue Apr 26, 2022 · 2 comments · Fixed by eveseat/web#635
Open

Feature Request - Cron schedule - Allow command argument entry #867

veteranmina opened this issue Apr 26, 2022 · 2 comments · Fixed by eveseat/web#635
Labels
bug complexity/medium This task has a medium complexity level core/jobs General things related to jobs flow

Comments

@veteranmina
Copy link

Currently in the schedule if a command has the optional argument parameter available this can be called in the background only via terminal.

In some circumstances one may wish to have more updates done i.e. notifications and other argument available commands. To do this would mean changing the frequency of the esi:update:notifications command which would cause unneeded taxed resources on both the SeAT instance and ESI. Allowing argument entry would reduce the amount of calling needed and focus on the character/corporation(s) selected and more flexibility for management of specific character/corporation(s).

@warlof warlof added packages/notifications Issues related to notification package or its dependencies. good first issue Good for newcomers core/jobs General things related to jobs flow complexity/low This task has a low complexity level and removed packages/notifications Issues related to notification package or its dependencies. labels Jun 8, 2022
@warlof
Copy link
Member

warlof commented Jun 8, 2022

@veteranmina normally it's already possible. Command drop-down will list you only available commands. However, if some of them might (or must) have ARGS, you can fill the drop-down list manually (directly using the search box).

As an example, type esi:update:notifications 73738374727 and select it when it's displayed in the list.

@warlof warlof added suport / help wanted and removed good first issue Good for newcomers core/jobs General things related to jobs flow complexity/low This task has a low complexity level labels Jun 8, 2022
@veteranmina
Copy link
Author

Attempting to add an argument i.e. (esi:update:notifications 96190383) and selecting to save produces the error message "The selected command is invalid." not allowing this.

@warlof warlof added bug core/jobs General things related to jobs flow complexity/medium This task has a medium complexity level and removed suport / help wanted labels Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug complexity/medium This task has a medium complexity level core/jobs General things related to jobs flow
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants