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
PL: teacher application form improvements #27011
Conversation
While the teacher application uses the PAGE_LABELS as helpers to get label strings, the details view used to view the application relies on PAGE_LABELS as the source of truth for which questions to show in which sections and in which order. As a way to preserve that source of truth, we can borrow a technique in which the client pulls across the label strings from another section.
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.
One small thing, looks good otherwise!
<ul> | ||
<li> | ||
<span style={styles.bold}>Section 1: About you</span> | ||
(Your and your principal’s contact info) |
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.
It looks like the principal's contact info is now in section 2
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.
Great catch. I missed this change in the spec.
Section3TeachingBackground, | ||
Section2ChooseYourProgram, |
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.
Note that I chose to not rename the classes & files for these, despite switching their order, just to reduce churn on the source code.
Before
section 1
section 2
section 3
thanks
After
section 1
section 1 - nominated
section 2
section 3
thanks