woocommerce / woocommerce-android

WooCommerce Android app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
275 stars 135 forks source link

[Woo POS] We don’t handle the low battery event during IPP flow properly #12565

Open backwardstruck opened 1 week ago

backwardstruck commented 1 week ago

LOW_BATTERY (same issue in the main app)

Image

dangermattic commented 1 week ago

🚫 Please add a type label (e.g. type: enhancement) and a feature label (e.g. feature: stats) to this issue.

backwardstruck commented 1 week ago

Should be something more like this:

Screenshot_20240909_124331

backwardstruck commented 1 week ago

It seems the resetConnectionState function is resetting the state to Unknown, which causes the LOW_BATTERY event to not be recognized as a Failed state. This doesn't happen for the LOW_BATTERY_SUCCEED_CONNECT because the connection is still maintained, hence the status isn't reset.