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

adapt_framework: move trackingIds to components from blocks #2805

Closed
oliverfoster opened this issue Jun 17, 2020 · 3 comments · Fixed by #3024
Closed

adapt_framework: move trackingIds to components from blocks #2805

oliverfoster opened this issue Jun 17, 2020 · 3 comments · Fixed by #3024
Assignees
@oliverfoster
Labels
enhancement

Comments

@oliverfoster
Copy link
Member

Subject of the issue

Blocks currently own the trackingIds for session state save and restore. This makes blocks mandatory as part of the course structure. It makes the component state save and restore more complicated than it needs to be.

By moving trackingIds to components we can remove the complexity and demote blocks from a necessary part of the course content to an optional part of the course content.
@oliverfoster oliverfoster self-assigned this Jun 17, 2020
This was referenced Jun 17, 2020
Merged
Merged
@oliverfoster
Copy link
Member Author

After assessment pr goes in adaptlearning/adapt-contrib-assessment#125 it needs updating to be tracking id location agnostic.

@moloko
Copy link
Contributor

moloko commented Aug 25, 2020

After assessment pr goes in adaptlearning/adapt-contrib-assessment#125 it needs updating to be tracking id location agnostic.

that PR has been merged now. want me to do a release of assessment before you add anything further?

@oliverfoster oliverfoster mentioned this issue Feb 1, 2021
Merged
@moloko moloko added this to Need fixing in The Quick TODO Board via automation Feb 15, 2021
@moloko moloko moved this from Need fixing to Need reviewing in The Quick TODO Board Feb 15, 2021
The Quick TODO Board automation moved this from Need reviewing to Need releasing Feb 22, 2021
@oliverfoster oliverfoster reopened this Feb 22, 2021
The Quick TODO Board automation moved this from Need releasing to Need fixing Feb 22, 2021
@oliverfoster oliverfoster removed this from Need fixing in The Quick TODO Board May 12, 2021
@oliverfoster oliverfoster added this to Assigned in Enhancements May 17, 2021
@oliverfoster oliverfoster added this to New in adapt_framework: The TODO Board via automation Jun 29, 2021
@oliverfoster oliverfoster moved this from New to Assigned in adapt_framework: The TODO Board Jun 29, 2021
@oliverfoster oliverfoster mentioned this issue Nov 29, 2021
Closed
Enhancements automation moved this from Assigned to Needs releasing Nov 29, 2021
adapt_framework: The TODO Board automation moved this from Assigned to Needs releasing Nov 29, 2021
@oliverfoster
Copy link
Member Author

moved to assessment plugin

@oliverfoster oliverfoster removed this from Needs releasing in Enhancements Nov 29, 2021
@oliverfoster oliverfoster removed this from Needs releasing in adapt_framework: The TODO Board Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oliverfoster oliverfoster

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

issue/2805 Added a mechanism for trackingId location agnosticism adaptlearning/adapt_framework
2 participants
@moloko @oliverfoster