Skip to content

CHANGE-ME link appears in system settings #1389

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

Closed
javmarina opened this issue Jul 8, 2018 · 5 comments
Closed

CHANGE-ME link appears in system settings #1389

javmarina opened this issue Jul 8, 2018 · 5 comments

Comments

@javmarina
Copy link

javmarina commented Jul 8, 2018

Step 2: Describe your environment

  • Android device: Nokia 7 plus
  • Android OS version: 8.1.0
  • Google Play Services version: 15.0.1 - 16.0.0
  • Firebase/Play Services SDK version: 16.0.0
  • FirebaseUI version: 4.1.0

Step 3: Describe the problem:

Steps to reproduce:

  1. Include FirebaseUI (auth) in a project
  2. Build and install
  3. Go to Settings > Apps > installed testing app > Open by default

Observed Results:

  • What happened?
    In the "Supported links" line, you can see CHANGE-ME as a link. This is because Firebase UI includes this in the manifest. CHANGE-ME is used for Github sign in, I think.

Expected Results:

  • What did you expect to happen?
    I didn't expect this to appear, as it is a string related to a part of the library that I don't use. Moreover, users can see it too.

screenshot_20180709-015538

@SUPERCILEX
Copy link
Collaborator

Ouch, that was a miscalculation on my part, sorry! 😊 For a more permanent solution, I think we might want to make people opt-in to the GitHub stuff by enabling the manifest entry, but for now, this should fix it:

<activity
    android:name=".ui.provider.GitHubLoginActivity"
    tools:node="remove" />

@javmarina
Copy link
Author

javmarina commented Jul 9, 2018

This is enough for now, thanks!

EDIT: don't know if I was doing something wrong or not, but I had to reinstall the app in order to make it work.

@noelmansour
Copy link

noelmansour commented Aug 20, 2018

I'm not sure if this affects priority at all, but this bug breaks app links (like those used for passwordless sign up).

Also, I had to put the fully qualified activity name for the workaround to work:

<activity
    android:name="com.firebase.ui.auth.ui.provider.GitHubLoginActivity"
    tools:node="remove" />

@samtstern
Copy link
Contributor

This was fixed and released in version 4.2.0

@javmarina
Copy link
Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants