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
Create report dialog #32382
Create report dialog #32382
Conversation
}; | ||
|
||
export class LessonStatusList extends Component { | ||
static propTypes = {}; |
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.
is this here in anticipation of the propTypes we'll need when we connect to real data?
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.
Yup! I figure we will at somepoint want to get the lesson list and status of the lessons from redux/props so I just left it there.
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.
Nice! These dialogs are really coming together!
Description
Creates a new component for the CreateStandardsReportDialog for the standards page in the progress tab of Teacher Dashboard. This is just created in storybook right now and not connected to anything. The component has two pages which are also components.
Also in order to reuse parts of the LessonStatusDialog I broke out at LessonStatusList to be used in both the LessonStatusDialog and CreateStandardsReportDialog.
Links
Testing story
Created a storybook entry for the component. Will add further testings once the component is connected to other components.
Reviewer Checklist: