search

4IceG / luci-app-sms-tool-js

Simple user interface to handle SMS / USSD / AT commands via sms_tool application. (LuCI JS) | OpenWrt >= 21.02
GNU General Public License v3.0
26 stars 15 forks source link

no indication of ongoing sms deletion process #22

Open bam80 opened 1 month ago

bam80 commented 1 month ago

If many SMS messages are deleting, the process seems takes time, but there is no indication if it's ongoing yet or finished. Messages count slowly decreases but this easily can be unnoticed.

If in the middle of the deletion process one presses "Refresh messages" button, the deletion stops.

This behavior confused me until I didn't find out what's going on here.

4IceG commented 1 month ago

A progress bar and a changing counter inform you about the progress of SMS deletion process. I have no influence on how the modem reacts to command to delete messages.

Question... who deletes and refreshes messages at the same time.

Tip: Don't delete 1000 messages at once, you will hang up modem.

bam80 commented 1 month ago

I see no progress bar on my side, and as I said changing counter is not helping here as it's unnoticeable. I delete just few tens of the messages, and yet got the problem.

Let's please reopen as it's clear issue here.

4IceG commented 1 month ago

I won't change it because it works for me (on my routers).

Another thing is that you are using a non standard theme (since you don't see the bar?) and your modem hangs when deleting a larger number of SMS (since the counter does not change).

bam80 commented 1 month ago

Thanks for reopening.

If you mean this bar, it doesn't help for tracking the deletion process, as it's not clear when it starts and ends: image I use luci-app-sms-tool-js 2.0.23-20240622.

If there is another bar: I'm not using anything non-standard and my modem doesn't hang, since the counter (on the screenshot) does change but it's simply unhelpful for the deletion process notion.

As an idea, some kind of additional (modal?) popup bar when the deletion starts would do the trick: it could be full from the start and represent messages count to be deleted, with the decreasing counter. After that, it could fully disappear.

4IceG commented 1 month ago

@bam80 Now you know what is missing in my package and what other SMS packages have, you can add missing functionality.

Writing a few lines of code in LuCI JS is trivial (probably).

At the moment, modifying package is not in my priorities, so you submit a PR with your own code or wait patiently.