marc136 / tonuino-nfc-tools

Android app to write NFC tags for the TonUINO diy music box
https://marc136.github.io/tonuino-nfc-tools
BSD 3-Clause "New" or "Revised" License
28 stars 9 forks source link

tag programming workflow #2

Closed bahnfrei closed 5 years ago

bahnfrei commented 5 years ago

while attaching new tags to the smartphone, the following dialog comes up, which is ok:

1-new

if you then select "Tag beschreiben" from the main screen, adjust values according to your needs and push the below button "Tag beschreiben", the following error dialog comes up:

2-error

it only works, if you first select "Tag beschreiben" from the main screen, secondly attach the new tag to the device, adjust settings and select the "new" button" "Daten auf Tag \<tagid> schreiben":

3-ok

bahnfrei commented 5 years ago

i figured out a similar issue while adjusting an already configured tag: it's working as long as the tagid is available/shown in the "Bearbeiten" menu, but sometimes, the tagid is missing, so the modification(s) wont be saved.

marc136 commented 5 years ago

Hello @bahnfrei, thanks for the feedback.

A problem with using NFC is that the tags don't have an open connection but are usually one-off read/write operations. So the connection to a tag might be lost at any time (I also noticed that when I put my phone on a tag, it will re-find that tag quite often..).

I will try to fix this in v0.3.1

if you then select "Tag beschreiben" from the main screen, adjust values according to your needs and push the below button "Tag beschreiben", the [no tag found] error dialog comes up

I'm trying to fix this by disabling the "write button" if no tag is known, see fc6b14.

it's working as long as the tagid is available/shown in the "Bearbeiten" menu, but sometimes, the tagid is missing, so the modification(s) wont be saved.

This might be partially fixed with the first change, but I will try to poll for the NFC tag and disable the button if the tag is out of range (with a small delay).

bahnfrei commented 5 years ago

Seems to be ok in 0.3.1#6, thx!