-
Notifications
You must be signed in to change notification settings - Fork 479
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
Standards toggle only shows for CSF courses #32430
Conversation
@@ -68,4 +70,16 @@ describe('SectionProgress', () => { | |||
); | |||
expect(wrapper.find('#uitest-detail-view').exists()).to.be.true; | |||
}); | |||
|
|||
it('standards view shows standards view and legend', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the name here seems a little confusing. The legend is part of the standards view right? Should it explicitly check for the legend or is it just checking for the whole standards view?
@@ -3,6 +3,8 @@ import {shallow} from 'enzyme'; | |||
import {expect} from '../../../util/deprecatedChai'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you update this to reconfigedChai while we are here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple small suggestions but looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after addressing @dmcavoy 's comments
spec
LP-957 Part 1
The new Standards view of the progress tab will only be applicable to CSF courses (Course A-F, Pre-Express and Express), therefore we need to hide that tab in the toggle for all other selections.
excludeCsfColumnInLegend
is already being used to distinguish CSF courses from other types of assignables, so I'm re-using it here to conditionally hide/show the Standards toggle.Up next is handling the case where the teacher switches from a CSF course -> non-CSF course while on the standards tab. Currently the standards tab disappears, but the content of the standards tab remains, instead the teacher should be taken to “Lessons” for the newly selected course.