Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Obsolete information regarding "whitelisting" for testers #5481

Closed
MikeMcC399 opened this issue Aug 22, 2022 · 5 comments
Closed

Obsolete information regarding "whitelisting" for testers #5481

MikeMcC399 opened this issue Aug 22, 2022 · 5 comments
Labels
bug Something isn't working enhancement Improvement of an existing feature

Comments

@MikeMcC399
Copy link
Contributor

MikeMcC399 commented Aug 22, 2022

In README: Known Issues the first paragraph is outdated:

"The Exposure Notification API is going to block you from successfully testing the Application unless you are whitelisted inside GMS; Shoutout to @pocmo for working on a demo mode to test unreleased app versions in advance without whitelisting (issue #321), if you want to contribute you can reach out here for the time being - thank you!"

The paragraph should be replaced by up-to-date information for testers.

@cwa-bot cwa-bot bot added this to ToDo in [CM] cwa-app-android Aug 22, 2022
@mtwalli
Copy link
Contributor

mtwalli commented Sep 6, 2022

@MikeMcC399 Do you suggest to keep only the sentence "The Exposure Notification API is going to block you from successfully testing the Application unless you are whitelisted inside GMS" ?

@mtwalli mtwalli added bug Something isn't working enhancement Improvement of an existing feature labels Sep 6, 2022
@MikeMcC399
Copy link
Contributor Author

@mtwalli

Do you suggest to keep only the sentence "The Exposure Notification API is going to block you from successfully testing the Application unless you are whitelisted inside GMS" ?

It does seem that the text "Shoutout to @pocmo for working on a demo mode to test unreleased app versions in advance without whitelisting (issue #321), if you want to contribute you can reach out here for the time being - thank you!" should be removed.

Leaving only "The Exposure Notification API is going to block you from successfully testing the Application unless you are whitelisted inside GMS" is only partially helpful. It should probably be added that whitelisting is only available for certain people and not for the community. Also, as I wrote in the OP, you don't need to be whitelisted in GMS to test functions which don't rely on the Exposure Notification API.

I can suggest a text for this if you want.

@mtwalli
Copy link
Contributor

mtwalli commented Sep 7, 2022

@MikeMcC399 Please add your suggestion? or if you want you can open a PR directly?

@MikeMcC399
Copy link
Contributor Author

@mtwalli

@mtwalli
Copy link
Contributor

mtwalli commented Sep 7, 2022

PR merged

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working enhancement Improvement of an existing feature
Development

No branches or pull requests

2 participants