-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
But we may already be a Beta Tester #3968
Comments
I actually don't see this in the dev branch I use... maybe the UI has changed and this is not needed anymore? This is actually a good first task for anyone who wants to take it: check whether the UI includes that string or not, in both master and the maintenance branch. Thanks! |
Maybe also related to #3976. |
A potential issue with the above string recommendation - |
OH, AND... do mention "alpha tester" too!
|
I want to work on this issue. |
@nicolas-raoul Should I start working on this issue? and also I need descriptions. |
I don't think there's any need to mention alpha in the app, as it's not possible to opt in to alpha solely within the app (it requires signing up to our google group). Alpha is generally for users who are already very familiar with our project and have looked us up on GitHub. It can be unstable and primarily exists as a shortcut to allow contributors to test master without having to do an AS build, so I don't feel comfortable recommending users to be alpha testers in-app.
IMO the description text already says "Opt in to our beta channel to get early access to new features etc". If there are currently no new releases available, then there is nothing to get "early access" to. @gouri-panda It seems like there is a PR up at #3984 ? @madisoncfallin are you interested in making the requested changes to your PR? @gouri-panda If you still are looking for an issue to work on, anything with the "bug" label can easily be assigned to you too. :) |
I would say do like |
We have a wiki for the same purpose: https://github.com/commons-app/commons-app-documentation/blob/master/android/Volunteers-welcome!.md |
OK, but
|
Hey! @misaochan @jidanni @nicolas-raoul |
@AdityaArunSharma The "good first issue" was about searching strings and it is done already, I just forgot to remove the tag, sorry for the confusion! Anyway welcome to the team, glad to have you on board ! :-) Would you mind finding another issue at https://github.com/commons-app/apps-android-commons/issues?utf8=%E2%9C%93&q=is%3Aopen+is%3Aissue+no%3Aassignee+-label%3Aassigned+-label%3Adebated+label%3Abug ? Thanks! |
@nicolas-raoul can you please tell me what actually we have to do in this issue? is this issue even relevant to be open? |
Beta versioning is not used as much as it used to be in the past, so no need to spend time on this now. |
In Settings > Feedback section:
Become a Beta Tester...
But we may already be a Beta Tester.
So the wording should say:
Become a Beta Tester (if not one already)....
Or else more fancy detection should be enabled.
The text was updated successfully, but these errors were encountered: