Skip to content

add courses that should have the report bug link in upper right#9663

Merged
sfilman merged 1 commit into
stagingfrom
addreportbug
Jul 22, 2016
Merged

add courses that should have the report bug link in upper right#9663
sfilman merged 1 commit into
stagingfrom
addreportbug

Conversation

@sfilman

@sfilman sfilman commented Jul 21, 2016

Copy link
Copy Markdown
Contributor

There are a number of courses that aren't beta or k5, but should still have the report a bug link in the upper right.

There are a number of courses that aren't beta or k5, but should still have the report a bug link in the upper right.
@sfilman sfilman assigned sfilman and davidsbailey and unassigned sfilman Jul 21, 2016
@sfilman

sfilman commented Jul 21, 2016

Copy link
Copy Markdown
Contributor Author

Hey @davidsbailey as trusty dotd, do you mind giving this one off PR a quick look?

@davidsbailey

Copy link
Copy Markdown
Member

LGTM, assuming we don't care about cspunit1, cspunit2, etc any more

@sfilman

sfilman commented Jul 22, 2016

Copy link
Copy Markdown
Contributor Author

Thanks ! @davidsbailey they already fall under the beta bucket so I just trimmed them for simplicity.

@sfilman sfilman merged commit 346c67c into staging Jul 22, 2016
@sfilman sfilman deleted the addreportbug branch July 22, 2016 01:33
@tanyaparker

Copy link
Copy Markdown
Contributor

FYI - We're going to add the report bug link to all courses (#9649). @sfilman Alice said it's ok to remove old csp units from beta. Can you confirm?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants