Closed goldyfruit closed 1 month ago
The changes introduce a new function, ask_optin()
, to the utils/common.sh
script, which prompts users for permission to upload log files to a specified URL. This function is integrated into the existing on_error()
function to enhance error handling during installation. Additionally, the formatting of the failure message has been updated, and minor adjustments were made to the apt_ensure()
function's loop syntax.
File | Change Summary |
---|---|
utils/common.sh | - Added function ask_optin() for user consent on log uploads. - Updated on_error() to call ask_optin() . - Adjusted failure message formatting. - Minor formatting change in apt_ensure() for-loop syntax. |
In the script where errors dwell,
A new prompt rings a gentle bell.
"May I upload?" it sweetly asks,
Ensuring privacy in our tasks.
With each change, we hop along,
Enhancing safety, making us strong! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Based on https://github.com/OpenVoiceOS/ovos-installer/pull/172#issuecomment-2406116747
Summary by CodeRabbit
New Features
Bug Fixes
Style