kolibri icon indicating copy to clipboard operation
kolibri copied to clipboard

Move core channel state into coach - the only place it is still used.

Open rtibbles opened this issue 1 year ago • 1 comments

Summary

  • Moves core channel state into the coach pluginModule

References

Fixes #12557

Reviewer guidance

Does the minimal fix to move the channel fetching logic from the core vuex state into coach. Will presumably be cleaned up in the future by other changes.


Testing checklist

  • [ ] Contributor has fully tested the PR manually
  • [ ] If there are any front-end changes, before/after screenshots are included
  • [ ] Critical user journeys are covered by Gherkin stories
  • [ ] Critical and brittle code paths are covered by unit tests

PR process

  • [ ] PR has the correct target branch and milestone
  • [ ] PR has 'needs review' or 'work-in-progress' label
  • [ ] If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
  • [ ] If this is an important user-facing change, PR or related issue has a 'changelog' label
  • [ ] If this includes an internal dependency change, a link to the diff is provided

Reviewer checklist

  • PR is fully functional
  • PR has been tested for accessibility regressions
  • External dependency files were updated if necessary (yarn and pip)
  • Documentation is updated
  • Contributor is in AUTHORS.md

rtibbles avatar Aug 20 '24 22:08 rtibbles