Skip to content
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

Add/user col user connected #3433

Merged
merged 6 commits into from Jun 21, 2016
Merged

Add/user col user connected #3433

merged 6 commits into from Jun 21, 2016

Conversation

dereksmart
Copy link
Member

This adds a blank column to the user admin table that will show a green Jetpack icon is the user is linked to a WordPress.com account.

jetpack-user-col

This PR is intended to provide a very simple functionality. Show a logo if user is linked. Tooltip on hover reads an explanation, currently: This user is linked and ready to fly with Jetpack.. Let's look at adding more functionality to this once the base 4.0 is merged in and we know how it'll tie in better with user connection stuff.

addresses #2601

we'll need this later, and it can't be in a private Class.
This adds a blank column to the user admin table that will show a green Jetpack icon is the user is linked to a WordPress.com account
@dereksmart dereksmart added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it General [Status] Needs Review To request a review from Crew. Label will be renamed soon. [Status] Needs Design Review Design has been added. Needs a review! labels Feb 21, 2016
@dereksmart dereksmart added this to the 4.0 milestone Feb 21, 2016
fill: #8cc258;
}
</style>
<? }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't use shorttags.

@jeherve
Copy link
Member

jeherve commented Feb 22, 2016

I made a couple more changes, including a link to "My Jetpack".

Would it be worth adding an emblem for the current user as well, but keep it grayed out if you're not connected to WordPress.com, with the same link to "My Jetpack"? It might be a good incentive for folks to link their own account as well.

@dereksmart
Copy link
Member Author

@jeherve good suggestions! How about we keep this branch as "base" functionality for now until 4.0 is further developed and we can implement more fanciness when we know how it will tie in better?

"My Jetpack" will disappear in Jetpack 4.0. Let's avoid linking to it from now on.
@see #3433 (comment)
@jeherve
Copy link
Member

jeherve commented Jun 21, 2016

This looks good to merge. 👍

@jeherve jeherve added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review To request a review from Crew. Label will be renamed soon. labels Jun 21, 2016
@jeherve jeherve merged commit d405fba into master Jun 21, 2016
@jeherve jeherve removed the [Status] Ready to Merge Go ahead, you can push that green button! label Jun 21, 2016
@jeherve jeherve deleted the add/user-col-user-connected branch June 21, 2016 10:27
jeherve added a commit that referenced this pull request Jun 21, 2016
"My Jetpack" will disappear in Jetpack 4.0. Let's avoid linking to it from now on.
@see #3433 (comment)
jeherve added a commit that referenced this pull request Jun 21, 2016
jeherve added a commit that referenced this pull request Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General [Status] Needs Design Review Design has been added. Needs a review! [Status] Requires String Changes [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants