Skip to content
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

Extra build variants (GPlay?FOSS?) #4

Closed
IzzySoft opened this issue Jul 14, 2022 · 23 comments
Closed

Extra build variants (GPlay?FOSS?) #4

IzzySoft opened this issue Jul 14, 2022 · 23 comments
Labels
enhancement New feature or request
Milestone

Comments

@IzzySoft
Copy link
Contributor

Hi @d4rken, I just wonder as my library scanner reports Bugsnag: is it opt-in, so I can override the Tracking marker for your app?

@d4rken
Copy link
Member

d4rken commented Jul 14, 2022

Hey how are you ☺️. Are looking to setup FDroid?

BugSnag SDK is currently just in there as it's part of my starter project, can't actually be enabled by the user atm (backend isn't setup).

Could maybe remove it.

Depends a bit on how popular this becomes.
If there is enough demand, I could release a donation version on Google Play, which would then warrant setting up BugSnag and hiring a designer for an app icon 😅.

Maybe a Build Variant for Google Play? With IAP and BugSnag, and a FOSS variant without? 🤔

@IzzySoft
Copy link
Contributor Author

Are looking to setup FDroid?

already did (in my repo).

BugSnag … can't actually be enabled by the user atm (backend isn't setup).

So it's not active … yet?

Could maybe remove it.

Then I of course can remove the Tracking AF (which looks a bit weird on an app like this 😉). If you promise to either remove or make it opt-in, I can override the automated flag straight away.

Maybe a Build Variant for Google Play? With IAP and BugSnag, and a FOSS variant without?

Sounds good to me! If you attach both APKs to releases, please let me know in advance so we can define which APK I can pick for my repo. And once you feel the app ready/mature enough for F-Droid.org, I can help with that as well.

@d4rken
Copy link
Member

d4rken commented Jul 15, 2022

Yeah you can remove the tag, I'll make it opt-in or move it to an extra build variant all together before setting it up.

Sounds good to me! If you attach both APKs to releases, please let me know in advance so we can define which APK I can pick for my repo. And once you feel the app ready/mature enough for F-Droid.org, I can help with that as well.

I'll keep the APKs on GitHub as is, and if there is a Google Play variant it will have to be APK bundles anyways and that's probably uninteresting to everyone but Google Play 😓

And once you feel the app ready/mature enough for F-Droid.org, I can help with that as well.

Awesome, give me a few more days 😉

@d4rken d4rken changed the title Bugsnag Extra build variants (GPlay?FOSS?) Jul 15, 2022
@d4rken d4rken added the enhancement New feature or request label Jul 15, 2022
@IzzySoft
Copy link
Contributor Author

IzzySoft commented Jul 15, 2022

Great, thanks! So the (only) APK attached here will be the one without Bugsnag then?

Awesome, give me a few more days

All the time you need. Your app is already available via my repo, so no need to hurry. Better start at F-Droid.org with something "mature" – "first impression" and all that 😉

PS: Tracking will be removed with the next sync around 6 pm UTC today.

@d4rken d4rken closed this as completed in 097d563 Jul 16, 2022
@d4rken
Copy link
Member

d4rken commented Jul 16, 2022

Created another build flavor, kept bugsnag in both.
Made the reporting opt-in for the github build and opt-out for gplay.
(Bugsnag is still not setup though until I actually do a Google Play release).

Relevant code for opt-in/out:
https://github.com/d4rken/mystic-permissions/blob/eea45b80655eeeded9d9a450514b26bcf95bb226/app/src/main/java/eu/darken/myperm/common/debug/autoreport/DebugSettings.kt#L19-L23

I added gplay billing dependency but it should only be included in the gplay buid. For the foss build it just links to GitHub.

There should still only be one APK attached to releases, but the name has slightly changed, e.g.:
eu.darken.myperm-v0.0.5(500)-FOSS-RELEASE-ea3d2e5.apk
does that matter?

@IzzySoft
Copy link
Contributor Author

Made the reporting opt-in for the github build and opt-out for gplay.

Sounds fine with me – so I can keep the Tracking AF out (will white-list Bugsnag for your app then)

I added gplay billing dependency but it should only be included in the gplay buid. For the foss build it just links to GitHub.

Again fine, thanks! TBH I do not care too much what you put into the Play build¹ as long as it does not hit the Foss build.

There should still only be one APK attached to releases, but the name has slightly changed

The name only matters if there are multiple APKs attached – because then I need to take care the updater does not pick the wrong one.

So thanks a lot – that's a very good solution for all involved!


¹ well, to a degree: would you add a bunch of invasive trackers, users might get curious whether they could really trust the Foss one or if something sneaked in there, too 🤣

@IzzySoft
Copy link
Contributor Author

PS: You forgot to attach the APK to the latest release 😱

@d4rken
Copy link
Member

d4rken commented Jul 16, 2022

PS: You forgot to attach the APK to the latest release scream

Ahhh thanks.

b10c120#diff-1a01768048176e9aae1daf1472171fd9134e697c1ed988f40d12370a483cd3ecR64

@IzzySoft
Copy link
Contributor Author

Gladly – and thanks to you! I just triggered the updater manually – fetched the update fine, so all seems set. Just an icon is not there (ah, yes, issue is already open for that I see).

As for metadata in general: would you like a PR shipping you initial fastlane structures? I could offer to send you what I've set up with my repo currently – so you could take over from there and adjust whenever needed. And build upon – e.g. adding screenshots, icon, featureGraphic 😉

@d4rken
Copy link
Member

d4rken commented Jul 18, 2022

FYI
@IzzySoft I've created a new GitHub org (https://github.com/d4rken-org) and transfered this project to it. I hope this does not cause issues for your build services? I think normal git just keeps working because GitHub forwards?

The idea is/was to make it easier to let people get involved as orgs have more fine grained permission control than repositories on personal accounts.

@IzzySoft
Copy link
Contributor Author

No build services here – and yes, redirects should work (at least for a while). If you can provide me a list of packageIds I should check, I can take care to update the URLs so everything is straight again.

Speaking of permissions and the current GH/MS controversy: should you move to Codeberg, just let me know. There are no forwards from here to there – but my updater of course can deal with Codeberg just fine (I've moved almost all of my repos there years ago and archived them here). And yes, you have orgs there as well as CI and pages 😉

@d4rken
Copy link
Member

d4rken commented Jul 18, 2022

If you can provide me a list of packageIds I should check, I can take care to update the URLs so everything is straight again.

What packageIds do you mean? Currently I just moved d4rken/mystic-permissions to d4rken-org/mystic-permissions.

If this is everything i hope it to be, I might move other apps of mine that are still on Google Play there (e.g. https://github.com/d4rken/bluemusic).

Codeberg

Seen it and I respect the effort, but so far MS has not made any moved that I found too concerning (except buying the whole thing). Quite happy with GitHub still 😄. Not sure I would go with Codeberg over Gitlab though 🤔

@IzzySoft
Copy link
Contributor Author

What packageIds do you mean?

Quick check: I can find eu.darken.bluemusic, eu.darken.myperm, eu.thedarken.rootvalidator and eu.thedarken.wldonate in my repo (links for eu.darken.myperm adjusted now).

Not sure I would go with Codeberg over Gitlab though

Codeberg is hosted in DE and is driven by a community (Codeberg e.V.) – both strong reasons for me. They have very good support – another strong argument. Also, migration works like a breeze. That I also have contact to some of the team is a bonus to me – but I very much like how it works. Well, just saying 😉

@d4rken
Copy link
Member

d4rken commented Jul 26, 2022

Is there a better crash tracking tool that wouldn't get tagged as Tracking when made opt-out?

@IzzySoft
Copy link
Contributor Author

Afraid not. It needs to be marked a false positive when it's opt-in. But be welcome to check potential candidates for "better otherwise" 😉

@IzzySoft
Copy link
Contributor Author

PS: your Readme has an PlayStore link that leads to a 404 – but no link to the alreay working representation in my repo. Mind to fix that? You know ehre to find a badge for that, right? 😉

And while on it: you could move that app banner to /fastlane/metadata/android/en-US/images/featureGraphic.png and place an icon.png next to it. I just picked the two manually – but if they were in Fastlane, my updater could take care for them automatically.

@d4rken
Copy link
Member

d4rken commented Jul 27, 2022

PS: your Readme has an PlayStore link that leads to a 404 – but no link to the alreay working representation in my repo. Mind to fix that?

Google has rejected the APK, need to submit it for review again.

You know ehre to find a badge for that, right? wink

I'm a bit confused on the different between yoru repos and F-Droid, could you elaborate?

And while on it: you could move that app banner to /fastlane/metadata/android/en-US/images/featureGraphic.png and place an icon.png next to it. I just picked the two manually – but if they were in Fastlane, my updater could take care for them automatically.

Good point, will do.

@d4rken d4rken added this to the v1.0.0 milestone Jul 27, 2022
@IzzySoft
Copy link
Contributor Author

Google has rejected the APK, need to submit it for review again.

Ah, the usual G-hell… 😢

I'm a bit confused on the different between yoru repos and F-Droid, could you elaborate?

Basically this. Plus that F-Droid's inclusion criteria are stricter.

Thanks for moving icon+featureGraphic to Fastlane – adjusted the config for my updater to check them on updates.

@d4rken
Copy link
Member

d4rken commented Jul 27, 2022

The badge doesn't fit nicely into the readme's current style, (or we get a matching badge for google play and github too) but I'm open to adding a bullet point to the download section.

Something like

  • Available for F-Droid via IzzyOnDroid repository

What happens when it is clicked? The F-droid app on the users phone opens?

@IzzySoft
Copy link
Contributor Author

The badge doesn't fit nicely into the readme's current style

Look more closely – and pick "the other one" which fits perfectly in the row between "release" and "downloads" 😉 Details on how to use that one are here. Further down you could then place a text link next to the others (or use the corresponding badges there for each of them: there's one for Google, and another for Github available, so the 3 would look similar in style to each other).

(I've just picked them from different locations – you may wish to copy them and keep them in your repo so they do not disappear)

What happens when it is clicked? The F-droid app on the users phone opens?

If you link it directly to the app details page, it will open in a browser. But you can use a slightly different URL: using https://apt.izzysoft.de/packages/eu.darken.myperm/, on the desktop it will be opened in the web browser (and redirected accordingly) – while on an Android device with the F-Droid app installed, the user will be prompted to choose between browser and F-Droid app.

@d4rken
Copy link
Member

d4rken commented Jul 27, 2022

Look more closely – and pick "the other one" which fits perfectly in the row between "release" and "downloads"

Yeah but that doesn't fit there, it's more for "look but not touch". The small badges do not present themselves to users as a primary way to download the app, especially when having 3 badges for 3 download places there 🤔

@d4rken
Copy link
Member

d4rken commented Jul 27, 2022

Maybe like this
https://github.com/d4rken-org/permission-pilot/blob/main/README.md#downloads

@IzzySoft
Copy link
Contributor Author

That looks fine! And even gives additional details. Cool!

d4rken pushed a commit that referenced this issue Aug 2, 2022
* Create ic_access_notification_policy_24.xml

Allows apps to read all your notifications, including personal information (such as contact names and text of messages you receive), snooze or dismiss notifications or trigger action buttons they contain (such as answer phone calls, delete your sms/ email, etc) and turn Do Not Disturb (DND) ON or OFF or change related settings.

* Update ic_access_notification_policy_24.xml

Allows the app to turn Do Not Disturb (DND) ON or OFF and change related settings.

* Create ic_access_notifications_24.xml

* Create ic_schedule_exact_alarm_24_xml

* Update APerm.kt

* Update strings.xml

* Rename ic_schedule_exact_alarm_24_xml to ic_schedule_exact_alarm_24.xml

* Update APerm.kt (#4)

* Update strings.xml

* Update APerm.kt

* Update APerm.kt

* Update APerm.kt

* Update APerm.kt

* Update APerm.kt

* Update strings.xml

* Update APerm.kt

* Update strings.xml

* Update APerm.kt

* Update strings.xml

* Update APerm.kt

* Update strings.xml

* Update APerm.kt

* Create ic_access_to_media_only_24.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants