-
-
Notifications
You must be signed in to change notification settings - Fork 16
Testing ChameleonMiniLiveDebugger #1
Comments
Ok, I will work on adding some of your suggestions. I have comp exams this week and next which are taking precedence over all for now. So it might be a little while before I get back into coding. I just added the two so as not to clutter up the already large tools menu interface too much. It’d easy to add them all. I’ll put it on my todo list. What do you think about the current APDU command recognition? Is it accurate enough, or still needs some improvement? |
Hi @maxieds |
Would you add the screen rotation bug to the issues page on my GitHub site
for the app so I remember to fix it? I'm sure I can have the app turn this
off by default so you don't have to.
Are you finding the apdu command recognition to be accurate, or do you
think it still needs work?
Maxie
…On Jan 9, 2018 3:08 PM, "init0" ***@***.***> wrote:
Hi @maxieds <https://github.com/maxieds>
I've performed some small tests.
Your app crashes when screen rotates - so i need to turn off screen
rotation before usage.
Logged 3 different connections and like the output of your log-export.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVI4uKOGNpvEz3EaAMKcIUaM1O9neHG7ks5tI8dWgaJpZM4RVP_m>
.
|
Sorry, I guess you already did this. I was expecting it to be from the CM issues site on my email. :) |
The screen rotation issue is fixed in the development version. I will roll out a new release on the Play Store later tonight (US Eastern time). |
Thanks. I'll test deeper if I find more time between work and family. |
The newest live version in the Play Store should incorporate several of your suggestions. It also fixes the screen rotation issue in the software without the user having to to it. Let me know what you think when you have some more time to test it out. @doegox: I noticed that you starred the repo and are located in Brussels, so I added a roll out in Belgium for you to use without having to proxy through an install. Please also give me your feed back on the new release. |
Excellent thank you @maxieds ! |
How do we see the result of a command? E.g. using the "(Almost) Complete Shell / LEDRED?" I see in the log tab "INFO: shell command of ledred? returned status 101:ok with text" but I don't see the expected text. EDIT: ok I understood: under some circumstances (bug) it repeats the last text instead of the new text, and in my first test it happened that it repeated an empty text. |
FYI for those having a phone with USB-C, I received today this cable https://www.amazon.fr/gp/product/B071H6ZW6Y and it works very nicely. |
The output of the command should be directly underneath the header (if
there is any to display). I found a couple more bugs, so I will release a
newer version this morning. I'm glad you're enjoying the newer release.
I still don't know how to fix the blank tabs when the app relaunches
because usb is plugged in. Something weird is going on with the way the USB
intent handlers are getting registered. It's on my list of bugs to fix, but
for now if it happens just restart the app.
Maxie
…On Jan 10, 2018 5:55 AM, "Philippe Teuwen" ***@***.***> wrote:
FYI for those having a phone with USB-C, I received today this cable
https://www.amazon.fr/gp/product/B071H6ZW6Y and it works very nicely.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVI4uFCblOZubl1AbllZOqFDNQhBJ2GTks5tJJcTgaJpZM4RVP_m>
.
|
A new release (0.0.8-v1) should be out soon. It features improved APDU command recognition and some bug fixes. Thanks for helping me test it! |
on the new version (0.0.8) peripherals and log mode, we don't see anymore the initial status of all leds & buttons. It was properly displayed in 0.0.6 |
I'm going to have to fix this later this afternoon. Let me know if the new
release broke anything else.
Maxie
…On Jan 10, 2018 10:26 AM, "Philippe Teuwen" ***@***.***> wrote:
on the new version (0.0.8) peripherals and log mode, we don't see anymore
the initial status of all leds & buttons. It was properly displayed in 0.0.6
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVI4uNj8i-w11Ejhe9NyQznbuSpN0Tbhks5tJNangaJpZM4RVP_m>
.
|
Here's a more recent screen shot of what the app looks like now. Thanks to @doegox for all of the suggestions. I'm going to roll out another release on play store once I figure out how to get this really annoying blank tabs after USB connect issue fixed. Look out for it tomorrow morning. |
I'm going to close the issue since it seems that all of the original requests have been handled. Please post new issue threads for future feature requests in the app. I'd like to see more of them. |
I’ll have done some first checks of your app.
It looks nice.
Some little things can be added:
Similar functions like in a serial terminal like this:
f.e. change settings, query actual setting, query actual config, ...
Would be great if you can add some like these functions.
(Here some example commands from my mobile terminal)
Why you only have under configure minimal peripherals only rbuttonlong and not the lbutton, lbuttonlong,...
#159
Cheers
The text was updated successfully, but these errors were encountered: