CUNY Academic Commons

I'm assuming that these items should be added under CUNY Academic Commons > Help, right? Do you want them displayed as a new flyout submenu (as CUNY Academic Commons > About) or using the dashes?

Yes, thanks. I think that the idea is to make the following options a fly-out from Help:

Help Site
- FAQs
- Codex
- Tour
WordPress Help Forum
Group for Group Admins

This, we would both be using a fly-out and using the dashes; the dashes help us avoid having to use a third-level fly-out. Does that make sense?

I added the links that are in the main menu to the admin bar, following the pattern of the "About" menu. I don't think it's ideal to hardcode the URL for the help site, but wasn't sure how to generate it in a way similar to bp_get_root_domain(). Also, on a second look I think I need to make a couple of formatting changes to bring things in line with Wordpress recommendations.

Here's a link to the changeset: https://github.com/cuny-academic-commons/cac/commit/1b192ecf517209ffb425e439b72b2f14753c902b.

Looks pretty good, Dan - thanks.

The hardcoded help.commons.gc.cuny.edu is not ideal, but it's not really worth doing something much more dynamic just for the sake of dev/staging sites.

One request I do have - both for http://help.commons.gc.cuny.edu and codex.commons.gc.cuny.edu - is to ensure that they are HTTP or HTTPS as appropriate (the Commons can be viewed over HTTPS, though we don't enforce it). Here's how to do it in WP:

$link = 'http://codex.commons.gc.cuny.edu';
if ( is_ssl() ) {
    $link = set_url_scheme( $link, 'https' );
}

Okay thanks Boone I added the SSL check for the help link and the Codex link here: https://github.com/cuny-academic-commons/cac/commit/4aa667a302d08ff123ddfe21898b34d2d738f2b6

I realized that I made a small typo in the Codex link which I fixed here - https://github.com/cuny-academic-commons/cac/commit/6e7b440ad390aafb7295978887b3ff0423c33cb3

Sorry I don't know how I didn't pick that up when I was testing the links before pushing. I'll be more careful next time. Is there anything else we should change here?