kiwix / kiwix-android-custom

Necessary data/tools for the Android custom apps
https://play.google.com/store/apps/developer?id=Kiwix+Team
GNU General Public License v3.0
14 stars 9 forks source link

DWDS custom app does not always find the ZIM on my phone #111

Closed benoit74 closed 11 months ago

benoit74 commented 11 months ago

On my phone, DWDS custom app fails to find the ZIM.

Here is what I've done:

image

This file appeared during the installation phase of the Play Store, I did not started the application yet.

Screenshot_20231020-083755

Logs: logs1697783883095.txt

Logs: logs1697784096523.txt

Same content in storage

There is nothing more in storage at /Android/data/org.kiwix.kiwixcustomdwds

Logs: logs1697784135913.txt

kelson42 commented 11 months ago

@MohitMaliFtechiz Related to https://github.com/kiwix/kiwix-android/issues/56 IMO. I suspect that fixing https://github.com/kiwix/kiwix-android/issues/56 will fix that one.

gouri-panda commented 11 months ago

I think this issue involves how/when we retrieve the asset. @MohitMaliFtechiz We should look into that.

After installation, we have the following data in storage: /Android/data/org.kiwix.kiwixcustomdwds/files/ Kiwix/ dwds_de_dictionnary_nopic_2023-09-12.zim (0 byte)

@MohitMaliFtechiz This is from the previous gradle task when we upload the expansion task. We can remove this, we can create file when we copying the asset. Thus we can remove the confusion. The other things i'm not sure. Maybe looking at the logs will help us.

MohitMaliFtechiz commented 11 months ago

I think this issue involves how/when we retrieve the asset. @MohitMaliFtechiz We should look into that.

@gouri-panda Yes, you are right. It is the problem of how we are retrieving the asset.

@benoit74 Thanks for providing the detailed information. I am able to reproduce the error, and placed a fix for it as you can see in below video.

https://github.com/kiwix/kiwix-android-custom/assets/34593983/d9c748fd-2d8e-4b2d-a6f2-63e090df8815

kelson42 commented 11 months ago

@MohitMaliFtechiz I don't know how many devices are impacted, but fixes seems straight. Please link with PR and make new release.