Skip to content

[fix] Fixed user defined commands that do not require input#871

Merged
nemesifier merged 1 commit into
masterfrom
fix-send-commands
Jun 13, 2024
Merged

[fix] Fixed user defined commands that do not require input#871
nemesifier merged 1 commit into
masterfrom
fix-send-commands

Conversation

@pandafy

@pandafy pandafy commented Jun 12, 2024

Copy link
Copy Markdown
Member

Bugs:

  • The "Send Command" operation was failing if the user defined command does not require an input.
  • The command.js was relying on presence of "reboot" default command. The JS would run into error in absence of the default "reboot" command and the loading overlay would not be hidden.

This patch fixes the above mentioned bugs.

@pandafy
pandafy force-pushed the fix-send-commands branch 2 times, most recently from d9fc627 to 79d044e Compare June 12, 2024 16:36
Bugs:
- The "Send Command" operation was failing if the user defined
  command does not require an input.
- The command.js was relying on presence of "reboot" default
  command. The JS would run into error in absence of the
  default "reboot" command and the loading overlay would
  not be hidden.

This patch fixes the above mentioned bugs.
@pandafy
pandafy force-pushed the fix-send-commands branch from 79d044e to 3011743 Compare June 12, 2024 17:21
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 98.752% (+0.001%) from 98.751%
when pulling 3011743 on fix-send-commands
into c3f2d7e on master.

@nemesifier
nemesifier merged commit 7aceba7 into master Jun 13, 2024
@nemesifier
nemesifier deleted the fix-send-commands branch June 13, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants