-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
APK issues #18
Comments
i will remove it later, thanks |
Remove what? |
The Internet permission |
Ah, thanks! Any plans for the other points listed above? |
Ah, I see you have the versioning fixed – great! So I've just enabled updates (the latest release should show up with the next sync around 6 pm UTC then). |
There are a few issues with the FOSS APKs attached to your releases:
1.2.5
and not the first one, yet the APK announces itself asversionCode='1' versionName='0.0.1'
.versionCode
must be increased with each release (as that's what Android uses internally to tell versions apart) – so as this is not the first release, the integer assigned there should be higher. AndversionName
should match what the tag claims it to be, here:1.2.5
INTERNET
though the description says "offline", it declaresandroid.permission.RECORD_AUDIO
but the description does not mention audio recording (it suggests its a note-taking app), it requestsandroid.permission.RECEIVE_BOOT_COMPLETED
andandroid.permission.FOREGROUND_SERVICE
though there's no hint on why those should be needed. The screenshots give no clue either. Could you please clarify?shot_description.txt
and missing ther
🙈I've just added your app to the IzzyOnDroid repo (it will show up here with the next sync around 6 pm UTC), and included an icon manually (taken from your Readme) as well as copied the "shot" description giving it the correct name – so be welcome to pick a badge to link there from your Readme. It would be nice to have above issues solved with the next release, though, so I can also enable automated updates here and your listing at F-Droid.org will work out, too 😉
For a "sneak preview":
The text was updated successfully, but these errors were encountered: