Closed simonchoxx closed 1 month ago
@darrensiegel Thanks for your review. I agree with what you say. I think it would just solve the problem of sharing necessary information between live components through URL params. Besides that it will help us (I think) to prevent data loss when refreshing the page. WDYT?
MER-3182
This PR adds two new functionalities to the tab that shows the metrics of the students per container.
A navigation component to browse through the containers and see the metrics of the students for each of them. You can choose to browse all containers or those that have been previously filtered in the
insights >> Content
tab.The button to go back to the Insights >> Content view, which can show
Back to Units
orBack to Modules
, will go back but show the results with the filters that had been applied at the time of entering the current tab.On the other hand, an ETS has been implemented, allowing to keep the state of the liveview and that both components can share information without losing it if a page reload happens.
https://github.com/Simon-Initiative/oli-torus/assets/16328384/ed41dd29-4d7f-492d-aa96-d260b0e7cfb3
https://github.com/Simon-Initiative/oli-torus/assets/16328384/7c9f4239-9a50-4959-bbe1-f52e7bacd33a