Skip to content

Changes for LMS-2591, Instructor Dash switchover - #3720

Merged
lamagnifica merged 1 commit into
masterfrom
ahodges/documentation/LMS2591
May 14, 2014
Merged

Changes for LMS-2591, Instructor Dash switchover#3720
lamagnifica merged 1 commit into
masterfrom
ahodges/documentation/LMS2591

Conversation

@lamagnifica

Copy link
Copy Markdown

@sarina, instructor dashboard changeover. (No screen shots were affected by this change! Minimal text changes needed.)
@mhoeber, @srpearce , please copy edit and proofread changes.

@srpearce

Copy link
Copy Markdown
Contributor

👍

@mhoeber

mhoeber commented May 14, 2014

Copy link
Copy Markdown
Contributor

👍 definitely want a change log entry for dashboard switch

@sarina

sarina commented May 14, 2014

Copy link
Copy Markdown
Contributor

@mhoeber it's the engineering team responsibility ot make a changelog entry, I thought? we already made one w/ the original PR.

@lamagnifica

Copy link
Copy Markdown
Author

@sarina I think @mhoeber is talking about the change_log.rst file that lives under the docs/en_us/course_authors branch

@sarina

sarina commented May 14, 2014

Copy link
Copy Markdown
Contributor

LGTM! 👍

Quick Q: do we have any documentation on anything that lives in the legacy dashboard?

@lamagnifica

Copy link
Copy Markdown
Author

@sarina no, we do not. Early on @shnayder and @mhoeber and I agreed that only New dash features would be documented. (I also searched the guide for "standard" to be sure)

@sarina

sarina commented May 14, 2014

Copy link
Copy Markdown
Contributor

Great! Just wanted to make sure :)

lamagnifica pushed a commit that referenced this pull request May 14, 2014
Changes for LMS-2591, Instructor Dash switchover
@lamagnifica
lamagnifica merged commit ac01da6 into master May 14, 2014
@lamagnifica
lamagnifica deleted the ahodges/documentation/LMS2591 branch May 14, 2014 16:55
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.

4 participants