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
Teachercon 18-19 Teacher Registration #19874
Conversation
9924e4b
to
afab7fb
Compare
908d5ba
to
e56ffdf
Compare
e56ffdf
to
a095e17
Compare
<p> | ||
We're excited you're planning to join us this summer! You will | ||
receive more information about travel approximately six weeks before | ||
TeacherCon. In the meantime, please <strong>do not</strong> book your flight, and make |
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 first 2 sentences are the same in all cases. Might it make sense to split that out into its own paragraph, supplied once?
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.
Actually, one says "join the Professional Learning Program" and the other two say "join us this summer" :/
I'mma just leave it as is; if we get lots of changes coming in to the copy I'll probably try and dedup to make those easier.
'teacherAcceptSeat', | ||
'teacherWaitlistExplain', | ||
'teacherDeclineExplain', | ||
] |
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.
This could be Object.keys(Joining.labels)
, if moved below the labels
definition.
<h4>Section 3: Releases</h4> | ||
<FormGroup> | ||
<ControlLabel> | ||
Please read this <a href="https://docs.google.com/document/d/12NVUcqccNRbVukoGMCserwSpg4vfG0vNlEqTxr6oit0/edit">photo release.</a> |
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.
Add target="_blank"
to open in a new window
</ControlLabel> | ||
{this.selectFor("photoRelease", { placeholder: "--" })} | ||
<ControlLabel> | ||
Please read this <a href="https://docs.google.com/document/d/15N5N1m-BPCU7obQDf7FLLhEt3IComFGB1u3N6kEQR6k/edit">liability waiver.</a> |
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.
same as above - new window
}; | ||
} | ||
|
||
//static sessionStorageKey = "Teachercon1819Registration"; |
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.
intentional?
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.
oh, thanks! I disabled it for testing and forgot to mark with TODO
// once they respond yes, but if we do that then the user-visible page count | ||
// will _grow_ as they progress through the form, which is a much weirder | ||
// user experience than it shrinking. | ||
if (!(this.state.data.teacherAcceptSeat && this.state.data.teacherAcceptSeat !== TeacherSeatAcceptanceOptions.accept)) { |
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 explanation :)
spec here
This PR only includes a fullly-implemented teacher version of the form; it will also be updated to support Facilitators and Partners, and a bit of the framework is already in place for that.
TODO:
Follow-up work (in separate PRs):