Skip to content

Documentation/new studio doc/12 04 13 - #1835

Merged
mhoeber merged 5 commits into
masterfrom
documentation/new_studio_doc/12-04-13
Dec 4, 2013
Merged

Documentation/new studio doc/12 04 13#1835
mhoeber merged 5 commits into
masterfrom
documentation/new_studio_doc/12-04-13

Conversation

@mhoeber

@mhoeber mhoeber commented Dec 4, 2013

Copy link
Copy Markdown
Contributor

New Studio documentation, includes Cale's LTI addition.

Can be seen at: http://draft-studio-documentation.readthedocs.org/en/latest/index.html

New documentation from rewrite, audit, and integration of some
documentation from edx101.  Intent to release by training on 12/5.
update to checking student progress, create new course, index, open
response assessment
Continued Editorial work
Worked through Andy's list
@mhoeber
mhoeber restored the documentation/new_studio_doc/12-04-13 branch December 4, 2013 16:11
@mhoeber

mhoeber commented Dec 4, 2013

Copy link
Copy Markdown
Contributor Author

@singingwolfboy and @stroilova can you check and give a thumbs up. I need it merged to generate new doc through read the docs.

@singingwolfboy

Copy link
Copy Markdown
Contributor

Looks great! 👍

In the future, I'd advocate for using the PNG format for all images, rather than GIF. It results in smaller file sizes, truer colors, and it's an open format whereas there's some ambiguity over just how "open" GIF is. However, it's not a big deal, and certainly shouldn't block this PR from going in.

@mhoeber

mhoeber commented Dec 4, 2013

Copy link
Copy Markdown
Contributor Author

agreed. the GIFs were taken from edx101 and in interest of time kept as
is. putting in a story to convert shortly.

On Wed, Dec 4, 2013 at 11:52 AM, David Baumgold notifications@github.comwrote:

Looks great! [image: 👍]

In the future, I'd advocate for using the PNG format for all images,
rather than GIF. It results in smaller file sizes, truer colors, and it's
an open format whereas there's some ambiguity over just how "open" GIF is.
However, it's not a big deal, and certainly shouldn't block this PR from
going in.


Reply to this email directly or view it on GitHubhttps://github.com/edx/edx-platform/pull/1835#issuecomment-29822489
.

@stroilova

Copy link
Copy Markdown
Contributor

Hi Mark,

I did not check everything in detail, but in general this LGTM 👍
I would be happy to look at it after it has compiled.

mhoeber added a commit that referenced this pull request Dec 4, 2013
@mhoeber
mhoeber merged commit 7b82de5 into master Dec 4, 2013
@mhoeber
mhoeber deleted the documentation/new_studio_doc/12-04-13 branch December 4, 2013 20:09
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Apr 24, 2017
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Apr 24, 2017
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Apr 24, 2017
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