Skip to content

Add help tokens: cohort courses & content libraries - #6633

Merged
mhoeber merged 1 commit into
masterfrom
carol/docs/helptokens
Jan 15, 2015
Merged

Add help tokens: cohort courses & content libraries#6633
mhoeber merged 1 commit into
masterfrom
carol/docs/helptokens

Conversation

@catong

@catong catong commented Jan 15, 2015

Copy link
Copy Markdown
Contributor

@mhoeber @andy-armstrong @marcotuts FYI. Updating config.ini to add help tokens for cohorted courseware and content libraries.
This PR adds 2 new help tokens, does not change any existing help tokens.

@mhoeber

mhoeber commented Jan 15, 2015

Copy link
Copy Markdown
Contributor

👍 I'm good with this. Want to confirm with @smagoun that this will make it's way to Saudi instance.

@catong

catong commented Jan 15, 2015

Copy link
Copy Markdown
Contributor Author

@mhoeber are you saying that you want me to wait until we confirm with @smagoun ?

@mhoeber

mhoeber commented Jan 15, 2015

Copy link
Copy Markdown
Contributor

@catong no, don't need to wait as there's no downside to merging this. just still want confirmation of saudi implementation.

If you want to go, I'll merge after tests complete.

@catong

catong commented Jan 15, 2015

Copy link
Copy Markdown
Contributor Author

@mhoeber it's OK. I'll do it; will check on it from home.

@smagoun

smagoun commented Jan 15, 2015

Copy link
Copy Markdown

@mhoeber @catong OK to merge - thanks! These should go into master so that other instances of Open edX can take advantage of them.

@mhoeber

mhoeber commented Jan 15, 2015

Copy link
Copy Markdown
Contributor

@smagoun merging, thanks. We'll need these for Birch release. Concern was more about getting to Saudi specifically before Birch is released

mhoeber added a commit that referenced this pull request Jan 15, 2015
Add help tokens: cohort courses & content libraries
@mhoeber
mhoeber merged commit 9d9c2af into master Jan 15, 2015
@mhoeber
mhoeber deleted the carol/docs/helptokens branch January 15, 2015 02:00
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