covid19cz / erouska-android

Fighting against COVID-19 via privacy-first Bluetooth tracing

https://erouska.cz

MIT License

202 stars 55 forks source link

Fix/and 231 init traveller val #535

Closed vandus closed 3 years ago

vandus commented 3 years ago

Description

Changed default traveller mode value to be enabled right off the bat. Fixed resetting its value when publishing the keys.

Screenshots 📸

How Has This Been Tested? 👨‍🔬

What Has Not Been Tested? 🙅🏻‍♂️

Not tested, trivial.

Checklist ✅

[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have checked all visual changes in both light and dark mode.

davidvavra commented 3 years ago

This also needs a different text on update screen - there is a big discussion about it on Slack.

vandus commented 3 years ago

@davidvavra do we need to wait with this PR until the discussion is finished? is it going to have affect on what I have done here?

davidvavra commented 3 years ago

@vandus The discussion is done and new texts are in Figma. I would prefer if new texts are in the same PR as the default value of the button, because it doesn't make sense to be by default ON with the existing texts.