Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

[WIP] Connect another service #6764

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[WIP] Connect another service #6764

wants to merge 3 commits into from

Conversation

vbudhram
Copy link
Contributor

@vbudhram vbudhram commented Dec 19, 2018

This was started from pair programming session with @philbooth at All-hands. The goal for this was to intelligently give users more services they can connect with their Firefox Account.

This screen would only be displayed if user can't login to Firefox Browser and is on a mobile device.

Feature doc: https://docs.google.com/document/d/1wjqiRT0pG97q5kMn7zU-3kDXkZ0pIzFqpN76fjC0dH4/edit#

cc @philbooth

@vbudhram vbudhram added the WIP label Dec 19, 2018
@ghost ghost assigned vbudhram Dec 19, 2018
@ghost ghost added the waffle:active label Dec 19, 2018
@vladikoff
Copy link
Contributor

You might be able to throw in Firefox Sync or Screenshots into the list in there as well? I understand that the main flow is already Sync, but still :)

@shane-tomlinson
Copy link

Overall I love it!

I find the green links confusing since green text is used for status messages elsewhere within FxA. The Photon style guide has a section for links that would be a good fit and be consistent with other links we have.

Since many of these apps are new, many users might not already know what they are. The descriptions help, but some users might want to find out more before signing up. Consider adding a link to the main site, maybe that could be done from the description, or the name.

@vbudhram
Copy link
Contributor Author

Other fixes came before this so it won't make this train, bumping.

@shane-tomlinson
Copy link

This repo has been deprecated and migrated to https://github.com/mozill/fxa. Please open this PR against that repo.

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

Successfully merging this pull request may close these issues.

4 participants