Skip to content

Conversation

@Aryan0826
Copy link

@Aryan0826 Aryan0826 commented Dec 21, 2025

This PR adds support for manually sending ACK messages from the SOTI CLI.

  • Adds an ack=true/false option to the send command
  • Stores the ACK flag in the Message object (default false)
  • Maintains compatibility with existing commands

This allows testing of satellite behavior under unexpected ACK scenarios.

Screenshot 2025-12-21 000006

@ArnavG-it
Copy link
Contributor

Nice work Aryan! The only issue I see is that you made your changes on the main branch. Usually we work off of the dev branch as that has the latest updates (like the new UI).

To fix that, it might be best to make a new branch based on dev, copy your changes over (you may have to tweak your code since dev is a little different), and then create a new PR for that branch.

@Aryan0826
Copy link
Author

Nice work Aryan! The only issue I see is that you made your changes on the main branch. Usually we work off of the dev branch as that has the latest updates (like the new UI).

To fix that, it might be best to make a new branch based on dev, copy your changes over (you may have to tweak your code since dev is a little different), and then create a new PR for that branch.

Thanks! I rebased the changes onto the dev branch and opened a new PR based on dev. Let me know if you'd like any adjustments.

@Aryan0826 Aryan0826 closed this Dec 24, 2025
@ArnavG-it ArnavG-it deleted the implement-sending-ack-messages-from-the-cli-#44 branch December 24, 2025 01:04
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