CorvetteCole / PixelWatchFace

A minimalistic and open-source watchface for WearOS
GNU General Public License v3.0
72 stars 15 forks source link

New onboarding messages don't scale correctly #45

Closed davwheat closed 4 years ago

davwheat commented 4 years ago

IMG_20200606_115029

CorvetteCole commented 4 years ago

thanks for the bug report, this is why I'm running it through beta before full release. I'll fix this by autoscaling text, thanks for the bug report!

CorvetteCole commented 4 years ago

@davwheat what watch is that? For testing purposes

davwheat commented 4 years ago

It's the Fossil Sport (4th Gen)

CorvetteCole commented 4 years ago

thanks!

CorvetteCole commented 4 years ago

@davwheat did any of the other onboarding dialogs present similar behavior or just this one? I just fixed this layout but may change the others if there are problems with them

davwheat commented 4 years ago

It was the same case with the others too.

CorvetteCole commented 4 years ago

got it, thanks again. Will fix all of these. Would you like to be made an internal tester of the app? Updates made to internal testing are much more likely to be broken but they are a lot faster for testing purposes. If no, I understand. If so shoot an email to support@corvettecole.com with your google account email and I'll add you to the internal testing group

davwheat commented 4 years ago

Pinged off an email! :)

CorvetteCole commented 4 years ago

I have pushed an update fixing this to the internal testing branch. You should see an update on your watch available immediately. To trigger the onboarding dialogs, reset the app data of the watch face (your settings will all be resynced when you trigger a change in the phone app). I will re-open this if you find it to still be an issue, thanks!

davwheat commented 4 years ago

Perfect!

davwheat commented 4 years ago

@CorvetteCole I did just notice that after the data clear, weather has disappeared. My data is still in the phone app, but I can't actually see it on the watch.

CorvetteCole commented 4 years ago

can you try changing a setting in the phone to re-sync your settings?

On Sun, Jun 7, 2020, 5:30 PM David Wheatley notifications@github.com wrote:

@CorvetteCole https://github.com/CorvetteCole I did just notice that after the data clear, weather has disappeared. My data is still in the phone app, but I can't actually see it on the watch.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CorvetteCole/PixelWatchFace/issues/45#issuecomment-640282736, or unsubscribe https://github.com/notifications/unsubscribe-auth/AESS5DNOSKFGSKHJ6PWM7ZLRVQBGXANCNFSM4NV3TGRQ .

davwheat commented 4 years ago

I've tried that already (as the WearOS logo wasn't showing either). My other settings (such as European date format) applied fine after clearing the watch app data.

CorvetteCole commented 4 years ago

So this new version also automagically sets some settings by default (24 hour time, European date format, Celsius temperature) based on your region and device settings. Is it not responding to any settings changes?

davwheat commented 4 years ago

It responds to my changes, but I cannot toggle weather on and off to "resync" it as it was disabled.

CorvetteCole commented 4 years ago

is the unlock now button present for the in-app purchase?

On Sun, Jun 7, 2020, 6:21 PM David Wheatley notifications@github.com wrote:

It responds to my changes, but I cannot toggle weather on and off as it was disabled.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CorvetteCole/PixelWatchFace/issues/45#issuecomment-640288540, or unsubscribe https://github.com/notifications/unsubscribe-auth/AESS5DM3XXDS6SURMDVKNNTRVQHH5ANCNFSM4NV3TGRQ .

davwheat commented 4 years ago

Yes, but my dark sky settings are still present.

CorvetteCole commented 4 years ago

what happens when you click it again? This purchase is non-consumable so it is impossible to be charged twice, don't worry.

On Sun, Jun 7, 2020, 6:26 PM David Wheatley notifications@github.com wrote:

Yes.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CorvetteCole/PixelWatchFace/issues/45#issuecomment-640289032, or unsubscribe https://github.com/notifications/unsubscribe-auth/AESS5DPE7X23WCKTCQ3CUSLRVQHZ5ANCNFSM4NV3TGRQ .

CorvetteCole commented 4 years ago

I think this might be related to becoming an internal tester. you should see test cards you can use, use the one that approves instantly. This may not be a bug at all

On Sun, Jun 7, 2020, 6:26 PM David Wheatley notifications@github.com wrote:

Yes.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CorvetteCole/PixelWatchFace/issues/45#issuecomment-640289032, or unsubscribe https://github.com/notifications/unsubscribe-auth/AESS5DPE7X23WCKTCQ3CUSLRVQHZ5ANCNFSM4NV3TGRQ .

davwheat commented 4 years ago

Ah, I thought it was going to remain free for existing weather users

If you have a Dark Sky API key saved when I release this next update, you will have access to the Dark Sky and weather settings

https://github.com/CorvetteCole/PixelWatchFace/issues/32#issuecomment-623020971

davwheat commented 4 years ago

No test cards appear -- I think those are only for you as you're the app developer.

CorvetteCole commented 4 years ago

oh you haven't purchased the weather unlock. I see. It would be quite involved to restrict my weather providers to ONLY use Dark Sky. The way I've built the system is so that with Dark Sky in use the other 5 weather providers are also available as fallbacks in case Dark Sky fails for whatever reason. Massive redundancy so we never have API issues again

On Sun, Jun 7, 2020, 6:30 PM David Wheatley notifications@github.com wrote:

Ah, I thought it was going to remain free for existing weather users

If you have a Dark Sky API key saved when I release this next update, you will have access to the Dark Sky and weather settings

32 (comment)

https://github.com/CorvetteCole/PixelWatchFace/issues/32#issuecomment-623020971

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CorvetteCole/PixelWatchFace/issues/45#issuecomment-640289459, or unsubscribe https://github.com/notifications/unsubscribe-auth/AESS5DJFM6GC64Y3JGBX6TDRVQIJBANCNFSM4NV3TGRQ .

CorvetteCole commented 4 years ago

Do you need a code to unlock the weather? I understand not everybody can purchase it

davwheat commented 4 years ago

No, don't worry. It's not a priority as I can still use the weather tile on WearOS. It's just a thing that's nice to have. Thanks for offering, though.