KamWithK / AnkiconnectAndroid

Unofficial Ankiconnect for Android (i.e. for Yomichan)
GNU General Public License v3.0
115 stars 14 forks source link

Allow requests that do not specify all fields #55

Closed killergerbah closed 6 months ago

killergerbah commented 7 months ago
killergerbah commented 7 months ago

Hey @KamWithK, sorry for the ping, but just wanted to make sure that this PR is on your radar, just in case it might have fallen through the cracks.

KamWithK commented 7 months ago

Hey @KamWithK, sorry for the ping, but just wanted to make sure that this PR is on your radar, just in case it might have fallen through the cracks.

Hey mate sorry for the late reply I largely reverse engineered yomichan and looked through the docs not the code so small things like this fell through the cracks

I've given it a quick test and cards with images get added, however the audio doesn't get added

killergerbah commented 7 months ago

Thanks for taking the time to test it out. I didn't have issues with audio myself but I'll see if I can reproduce and fix it.

KamWithK commented 7 months ago

Thanks for taking the time to test it out. I didn't have issues with audio myself but I'll see if I can reproduce and fix it.

Sounds good I tested on the word 商業 btw

killergerbah commented 7 months ago

I was not able to reproduce the issue using Yomitan. I tried making a couple cards, one of them being 商業 and the audio field was populated, and played fine from AnkiDroid. I inspected the logic again and I couldn't there don't seem to be errors from just reading through it. Is there anything else specific to your setup that might be causing the issue?

KamWithK commented 6 months ago

I was not able to reproduce the issue using Yomitan. I tried making a couple cards, one of them being 商業 and the audio field was populated, and played fine from AnkiDroid. I inspected the logic again and I couldn't there don't seem to be errors from just reading through it. Is there anything else specific to your setup that might be causing the issue?

I don't think there's anything in my setup which would cause it I'll give it another go and see if it was just a temporary issue because some of my stuff was very outdated

killergerbah commented 6 months ago

Sweet thanks!