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

Fix sections assigned #3744

Merged
merged 5 commits into from
Sep 8, 2021
Merged

Fix sections assigned #3744

merged 5 commits into from
Sep 8, 2021

Conversation

jivey
Copy link
Member

@jivey jivey commented Sep 1, 2021

This fixes a bug when the plan ecosystem doesn't match the course ecosystem — the reference book was being fetched with the course's ecosystem, instead of the plan's ecosystem.

Also fixes a mobx warning by wrapping the observable change in an action.

@jivey jivey self-assigned this Sep 1, 2021
@jivey jivey requested review from a team and Dantemss and removed request for a team September 1, 2021 21:50
@jivey jivey requested a review from a team as a code owner September 8, 2021 16:31
@jivey jivey merged commit 1d6ee6e into main Sep 8, 2021
@jivey jivey deleted the fix-sections-assigned branch September 8, 2021 17:26
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.

2 participants