Skip to content

Update TrainingProviderList to use Apollo

Samantha Ming requested to merge refactor-training-graphql into master

What does this MR do and why?

Previously, we made the apollo query call in the App.vue component and we pass the responding date into the TrainingProviderList.vue component. However in this next issue, #347414 (closed), we will need to add mutation logic, which will expand the App.vue file even larger. The file is already quite intensive to begin with.

So we're moving all the apollo logic into the TrainingProviderList.vue, this will significantly improve readability, make it easier to write tests, and avoid expanding the App.vue unneedlessly, yay 👏

This MR is part of the new security training feature, please see the epic for details and the implementation breakdown > &6176 (closed)

Screenshots or screen recordings

This MR does not introduce any UI changes

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Samantha Ming

Merge request reports