Teach-1865/modularity support in section progress#66214
Conversation
8ab1eca to
c28e67c
Compare
52eed26 to
f5cbab8
Compare
292b6df to
74c5211
Compare
sorry for the basic question, but what does "current" or "currently selected" course mean in this case? On the progress view in teacher dashboard, I do not see any course in the url, for example. |
Currently selected is the course that is selected in the unit dropdown. When you select a lesson in the progress table, it brings you to that lesson page. example.movBefore this PR, it would always bring you to the original course. Screen.Recording.2025-07-17.at.1.36.21.PM.mov |
davidsbailey
left a comment
There was a problem hiding this comment.
nice tests! just a couple nits, otherwise looks good
There was a problem hiding this comment.
I think we will want to log courseId here too:
| expect(getSelectedScriptDescription(state)).toEqual(null); | ||
| }); | ||
| }); | ||
| }); |
There was a problem hiding this comment.
leave describe('getSelectedScriptDescription' outside of describe('getSelectedUnitName'?
There was a problem hiding this comment.
ah yes, good catch. there were a couple of weird nesting things going on in this file and I must've missed this one. Thank you
Depends on #66364
Right now, the links on the progress page always defaults to the original course URL instead of the current course's URL. This PR updates the method to use the currently selected course instead.
Links
Jira: TEACH-1865
Testing story
Manually verified and updated unit/UI tests