-
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
Navigate to script overview on save and close of lesson edit for lessons without lesson plan #38688
Conversation
…script overview on close
); | ||
if (data.hasLessonPlan) { | ||
navigateToHref( | ||
`/lessons/${this.state.originalLessonData.id}${ |
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.
please pass the lesson path down from the server using the lesson_path
helper. this will make it a lot easier to find and change the lesson paths later.
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.
I will create a follow up PR with this
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.
If you set has_lesson_plan to false on the lesson edit page and then hit "save and close" it will take you to the script overview page instead of the lesson plan page.
You also can't directly access a lesson plan for a lesson for which has_lesson_plan is false.
Links
Testing story
Reviewer Checklist: