initial fastlane structures from IzzyOnDroid#302
Conversation
|
Cheers for this, seems like a good way of keeping things in sync. Think I'll take the opportunity to move a few bits around - will aim to get to it this weekend. |
|
I've suffered from a certain amount of scope-creep here but have just pushed an updated version of this:
In general, does this updated version look okay to you? Hopefully I've not made any mistakes with my changes 😄 |
In general it looks fine, except for those two glitches (locale, fulldesc formatting) 😊 |
Add initial fastlane format metadata with short/full description values imported from IzzyOnDroid and move the feature graphic and icon images originally created for the Play Store to the appropriate paths.
|
Cheers for the rapid feedback 😄. I've reversed my overoptimistic locale change, cheers for the explanation on that one. For In theory that means this should be ready to merge now but I'll leave it for a day or two to give you the chance to shout if there's anything else I've messed up 😆 Also, this isn't really the right forum (would the correct one be https://gitlab.com/IzzyOnDroid/applists/-/issues ?) and is a very minor thing but the app license is |
|
Oh, and also on the subject of metadata, should the app's |
|
Thanks Matt! Let me shortly walk the points:
That said: I cannot say anything on the Ruby script there, but the fastlane part looks fine. Though I miss some screenshot(s). Are there any (1 or 2 would be fine)? Last, but we can do that separately: Be welcome to pick a badge for your Readme to link to your app at IzzyOnDroid:
|
Oops, missed to answer that one. Close, but no cigar. It would be "repo", not "applist". The latter is for https://android.izzysoft.de/applists 😉 |
Change from the original imported full description value to a copy of the relevant content from README.md. Add a blank line to both before the actions bullet points for wider markdown parser compatibility. As ending a line with a backslash is a non-standard extension, switch this to two spaces in the full description. To prevent either the full or short description values getting out of sync with the README, add a new Overcommit hook to check that they match if any of them are updated.
|
Thanks for the heads-up on the permissions section not being visible in the clients - have changed tack and reinstated that section but have replaced the backslashes with double spaces to make it (hopefully) universally compatible. AntiFeature - cheers for the explanation, that makes complete sense and am very happy to leave it as-is 👍. License - all of the source files in the repo should have Screenshots - my issue here is that the app doesn't really have much of a UI to speak of, it's got a handful of fairly empty settings pages, an update screen that's only shown if the user chooses to check for updates and tap on the notification and a lot of hidden 'plumbing' to set up the correct environment to run rsync and allow it to find dbclient etc. I could add a screenshot of the key generation and rsync config screens (which probably have the most going on) if you think they would be less off-putting than no screenshots at all 😆
Ha, seems so obvious now you've pointed out my mistake 😂. Thanks for all of your help and feedback on this one, much appreciated. I'll merge this once the checks have completed and can apply any other tweaks needed after the next release 👍. |
Oof, I just looked at the Readme, and what Github claimed. You're of course right, those SPDX should suffice! Still helps to get it punched right on the nose, if you'd add it to the Readme – that then squashes the tomatoes on the eyes for those as blind as me 🙈
Totally up to you! We're not at PlayStore here where you're forced to provide at least 2 (we all know what that lead to). If there are none, none are needed. If there's at least a settings page or such, that helps visualizing what to expect. We don't want pictures to have pictures – but to be helpful to those browsing the catalogue 😉
Yes, that sounds great! Please do.
Gladly, anytime, and any time again 🤗 Thanks for merging! |
|
Ah, thanks for the heads-up! I had processed too many issues these days. In the evening I thought 💩 some license adjustment, where was it 🙈 Done now. Screenshots would be pulled along with the next release. If you feel it needed, I can trigger a fastlane sync before, so they'd go live within 24h. |
A sync would be great if it isn't too much trouble, thank you 🙂 |
Done. |
|
Awesome, thank you 😄. I can see that the description has updated (and the markdown has converted as intended 🎉) but I can't see the screenshots on the app page, does that need an extra nudge from your side? |
Quoting from the initial post:
In your case, those are the screenshots 😉 Checked, look fine, enabled, manually triggered a sync… should show up with the next repo sync around 6 pm UTC now. Thanks for the nudge – and for adding screenshots of course! 🤗 |
this PR provides you with a "fastlane starter package" – to give it into your hands to define how your app is presented, and make it easy to keep descriptions & graphics in sync with your development. Some notes to sum them up in a place easy to find for you:
full_description.txtmax 4,000 chars, graphics must have specific aspect ratios etc.full_description.txthere I've used HTML compressed into a single line, to prevent the fdroid software converting each line break to a<br>. The tags used for this are supported by F-Droid.org as well, and to my knowledge even by PlayStore.A second commit adds a link to IzzyOnDroid to the Readme, where TaskerRsync is available since 2022-10-06 – and congrats, it's confirmed as reproducible build (see e.g. Reproducible Builds, special client support and more at IzzyOnDroid for some background on this) 🥳 So if you wish, there's also a shield available showing the last known RB status:
[<img src="https://shields.rbtlog.dev/simple/com.nerdoftheherd.tasker.rsync" alt="RB shield">](https://shields.rbtlog.dev/com.nerdoftheherd.tasker.rsync)And now: Enjoy!