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.

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

Fetch completion info from user progress bloc #351

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

JElgar
Copy link
Member

@JElgar JElgar commented Nov 1, 2024

Description

Currently we store completion info inside resource models. This make its pretty hard to update after users complete more stuff which is what #350 is about. This pulls all completion/joined data from user progress bloc which is updated after each resource is completed and is therefore easy to keep up to date.

Fixes #350

Checklist:

Creator

  • The target branch is main
  • I have updated the unreleased section of the change log
  • I have reviewed the 'files changed' tab on github to ensure all changes are as expected
  • I have added someone to review the pr

Reviewer

  • I have verified the above are all completed
  • I have run the code locally to ensure it fufills the requirements of the issue
  • I have reviewed the 'files changed' and commented on any sections which I think are not needed, incorrect or could be improved

After pull

  • If appropriate I have closed the issue/ moved the trello card

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

[BUG] - Learning actions in campaigns are not shown as completed until you leave and revisit the page.
1 participant