-
-
Notifications
You must be signed in to change notification settings - Fork 280
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
feat: 4510 - new "enroll in the beta" contribute item #4938
feat: 4510 - new "enroll in the beta" contribute item #4938
Conversation
Impacted files: * `app_en.arb`: added an "enroll in the beta" label. * `app_store.dart`: new `getEnrollInBetaURL` method * `apple_app_store.dart`: implements new `getEnrollInBetaURL` method * `google_play.dart`: implements new `getEnrollInBetaURL` method * `user_preferences_contribute.dart`: added an "enroll in the beta" button
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #4938 +/- ##
==========================================
+ Coverage 9.65% 9.66% +0.01%
==========================================
Files 323 322 -1
Lines 16207 16197 -10
==========================================
+ Hits 1564 1565 +1
+ Misses 14643 14632 -11 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- https://fonts.google.com/icons?selected=Material+Symbols+Outlined:labs:FILL@0;wght@400;GRAD@0;opsz@24&icon.query=test&icon.platform=android
- Mention "internal alpha version" and not just beta. Probably a bottom sheet to get the user to acknowledge that: complete loss of data is possible, the apps may become unusable at any time.
@teolemon Instead of a bottom dialog, what about this: Regarding the icon, it looks like it's not accessible with flutter (cf. https://fonts.google.com/icons?icon.query=labs). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer the red warning to be in a model or dialog to not draw all attention but codewise good 👍🏼
@M123-dev I'll have to code a bit anyway, my screenshot was just a suggestion without a pushed code. |
Co-authored-by: Pierre Slamich <[email protected]>
@teolemon For the record there's no space before an exclamation mark in English. |
@teolemon If it's just a matter of filling the test tube that can be done: |
That looks more in line with the other icons 👍 |
packages/smooth_app/lib/pages/preferences/user_preferences_contribute.dart
Outdated
Show resolved
Hide resolved
All requested changes are there, now.
What
Screenshot
Fixes bug(s)
Impacted files
app_en.arb
: added an "enroll in the beta" label.app_store.dart
: newgetEnrollInBetaURL
methodapple_app_store.dart
: implements newgetEnrollInBetaURL
methodgoogle_play.dart
: implements newgetEnrollInBetaURL
methoduser_preferences_contribute.dart
: added an "enroll in the beta" button