Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

the setting _requireCompletionOf is not exposed to the authoring tool #2639

Closed
moloko opened this issue Jan 29, 2020 · 4 comments
Closed

the setting _requireCompletionOf is not exposed to the authoring tool #2639

moloko opened this issue Jan 29, 2020 · 4 comments
Assignees
@jamesrea83

Comments

@moloko
Copy link
Contributor

moloko commented Jan 29, 2020

original implementation of this is here: #549 (useful for the original discussion around why this was added)

actual implementation was in #951

Ideally support should be backported to v4 - and possibly v3?
@moloko
Copy link
Contributor Author

moloko commented Jan 29, 2020

If anyone wants to pick this up it would be great to sort out #1912 at the same time...

@moloko
Copy link
Contributor Author

moloko commented Jan 29, 2020

#1657 also!

@jamesrea83 jamesrea83 self-assigned this Feb 19, 2020
jamesrea83 added a commit to jamesrea83/adapt_framework that referenced this issue Feb 19, 2020
@jamesrea83
adaptlearning#2639 expose _requireCompletionOf for course / contentOb…
320cb58 
…ject / article / block to authoring tool
@jamesrea83 jamesrea83 mentioned this issue Feb 19, 2020
Merged
jamesrea83 added a commit that referenced this issue Feb 19, 2020
@jamesrea83
#2639 expose _requireCompletionOf for course / contentObject / articl…
ae2f195 
…e / block to authoring tool (#2653)
@jamesrea83 jamesrea83 mentioned this issue Feb 19, 2020
Closed
jamesrea83 added a commit that referenced this issue Apr 2, 2020
@jamesrea83
#2639 expose _requireCompletionOf for course / contentObject / articl…
80bc24b 
…e / block. Opening a new PR due to a messy update.
@jamesrea83 jamesrea83 mentioned this issue Apr 2, 2020
Merged
@olivermartinfoster
Copy link
Contributor

is this done now?

@moloko
Copy link
Contributor Author

moloko commented May 7, 2020

Yes I think so?

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

@jamesrea83 jamesrea83

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@moloko @olivermartinfoster @jamesrea83