More bug fixes for TH 25

pages/teams.tsx

@@ -150,53 +150,56 @@ export default function ViewTeams(props: SSRDataAuth<TeamData>['props']) {
 			<Menu />
 			<WaveHeader variant="light" />
 			<div>
+
 				<WaveFooter />
-				<FloatingDiv>
-					<ContentHeader title="Team" />
-					{!loading ? (
-						<form
-							className={styles.buttonForm}
-							onClick={async (e) => {
-								e.preventDefault();
-								router.push('/teams/create');
-							}}
-						>
-							<RectangleButton
-								type="submit"
-								className={styles.newTeamButton}
+				<div className={styles.dialog}>
+					<FloatingDiv>
+						<ContentHeader title="Team" />
+						{!loading ? (
+							<form
+								className={styles.buttonForm}
+								onClick={async (e) => {
+									e.preventDefault();
+									router.push('/teams/create');
+								}}
+							>
+								<RectangleButton
+									type="submit"
+									className={styles.newTeamButton}
+								>
+									Create new team
+								</RectangleButton>
+							</form>
+						) : null}
+						<Collapse in={loading}>
+							<div className={styles.spinnerContainer}>
+								<CircularProgress />
+							</div>
+						</Collapse>
+						<div className={styles.tableHeader}>
+							<Typography
+								variant="h4"
+								className={styles.tableHeaderText}
 							>
-								Create new team
-							</RectangleButton>
-						</form>
-					) : null}
-					<Collapse in={loading}>
-						<div className={styles.spinnerContainer}>
-							<CircularProgress />
+								Open Teams
+							</Typography>
 						</div>
-					</Collapse>
-					<div className={styles.tableHeader}>
-						<Typography
-							variant="h4"
-							className={styles.tableHeaderText}
-						>
-							Open Teams
-						</Typography>
-					</div>
-					{emptyMessage}
-					<table className={styles.tableData}>
-						<tbody>
-							{teams
-								.sort((a, b) => +a._id - +b._id)
-								.map((team, idx) => (
-									<TeamTableEntry
-										team={team}
-										key={idx}
-										callback={checkJoinErrorCallback}
-									/>
-								))}
-						</tbody>
-					</table>
-				</FloatingDiv>
+						{emptyMessage}
+						<table className={styles.tableData}>
+							<tbody>
+								{teams
+									.sort((a, b) => +a._id - +b._id)
+									.map((team, idx) => (
+										<TeamTableEntry
+											team={team}
+											key={idx}
+											callback={checkJoinErrorCallback}
+										/>
+									))}
+							</tbody>
+						</table>
+					</FloatingDiv>
+				</div>
 				<Snackbar
 					open={notify === 'error' || notify === 'success'}
 					autoHideDuration={5000}

pages/teams/details/[teamId].tsx

@@ -182,7 +182,7 @@ const TeamDescription = () => {
 		<>
 			<Menu />
 			<WaveHeader variant="light" />
-			<div>
+			<div className={styles.dialog}>
 				<WaveFooter />
 				<FloatingDiv>
 					{ownTeamFetched && !isOwnTeam ? (

src/actions/user.ts

@@ -92,7 +92,6 @@ export const rejectAll = (): DispatchAction => ({
 
 export const confirm = (
 	signatureLiability: boolean,
-	signatureCodeOfConduct: boolean,
 	willMentor: boolean,
 ): DispatchAction => ({
 	type: DispatchActionType.USER_CONFIRM,
@@ -102,7 +101,6 @@ export const confirm = (
 		method: 'PUT',
 		body: {
 			signatureLiability,
-			signatureCodeOfConduct,
 			willMentor,
 		},
 	},

src/components/admin/AdminDialog/index.module.scss

@@ -1,6 +1,14 @@
 @import '~/styles/variables.scss';
 @import '~/styles/mixins.scss';
 
+.dialog {
+	width: 100%;
+	height: 100%;
+	display: flex;
+	align-items: center;
+	justify-content: center;
+}
+
 .tabs {
 	width: 100%;
 	justify-content: space-between;

src/components/admin/AdminDialog/index.tsx

@@ -28,34 +28,36 @@ const AdminDialog = (): ReactElement => {
 	}, []);
 
   return (
-    <FloatingDiv>
-      <TabContext value={tabIndex}>
-        <Box>
-          <TabList
-            onChange={(e, newIndex: string) => setTabIndex(newIndex)}
-            className={styles.tabs}
-            variant="scrollable"
-            >
-            <Tab label="Participants" value="0" />
-            <Tab label="Recruiters" value="1" />
-            <Tab label="Sponsors" value="2" />
-            <Tab label="Analytics" value="3" />
-          </TabList>
-        </Box>
-        <TabPanel value="0" className={styles.tabPanel}>
-          <ParticipantTable />
-        </TabPanel>
-        <TabPanel value="1" className={styles.tabPanel}>
-          <RecruiterCreationForm />
-        </TabPanel>
-        <TabPanel value="2" className={styles.tabPanel}>
-          <SponsorCreationForm />
-        </TabPanel>
-        <TabPanel value="3" className={styles.tabPanel}>
-          <AnalyticsTab />
-        </TabPanel>
-      </TabContext>
-    </FloatingDiv>
+    <div className={styles.dialog}>
+      <FloatingDiv>
+        <TabContext value={tabIndex}>
+          <Box>
+            <TabList
+              onChange={(e, newIndex: string) => setTabIndex(newIndex)}
+              className={styles.tabs}
+              variant="scrollable"
+              >
+              <Tab label="Participants" value="0" />
+              <Tab label="Recruiters" value="1" />
+              <Tab label="Sponsors" value="2" />
+              <Tab label="Analytics" value="3" />
+            </TabList>
+          </Box>
+          <TabPanel value="0" className={styles.tabPanel}>
+            <ParticipantTable />
+          </TabPanel>
+          <TabPanel value="1" className={styles.tabPanel}>
+            <RecruiterCreationForm />
+          </TabPanel>
+          <TabPanel value="2" className={styles.tabPanel}>
+            <SponsorCreationForm />
+          </TabPanel>
+          <TabPanel value="3" className={styles.tabPanel}>
+            <AnalyticsTab />
+          </TabPanel>
+        </TabContext>
+      </FloatingDiv>
+    </div>
   )
 }
 

src/components/confirmation/ConfirmationDialog/index.tsx

@@ -3,6 +3,7 @@ import {
 	Collapse,
 	FormControlLabel,
 	FormGroup,
+	Hidden,
 	IconButton,
 	LinearProgress,
 	Link,
@@ -26,7 +27,6 @@ const ConfirmationDialog = (): ReactElement => {
 	const [loading, setLoading] = useState(false);
 
 	const [signatureLiability, setSignatureLiability] = useState(false);
-	const [signatureCodeOfConduct, setSignatureCodeOfConduct] = useState(false);
 	const [willMentor, setWillMentor] = useState(false);
 
 	const confirmTime = useSelector(
@@ -43,7 +43,6 @@ const ConfirmationDialog = (): ReactElement => {
 			await dispatch(
 				actions.user.confirm(
 					signatureLiability,
-					signatureCodeOfConduct,
 					willMentor,
 				),
 			);
@@ -75,6 +74,7 @@ const ConfirmationDialog = (): ReactElement => {
 					<div className={styles.dialogText}>
 						<FormGroup className={styles.formGroup}>
 							<FormControlLabel
+								sx={{ ".MuiFormControlLabel-asterisk": { display: "none" } }}
 								control={
 									<Checkbox
 										required
@@ -100,32 +100,6 @@ const ConfirmationDialog = (): ReactElement => {
 									</Typography>
 								}
 							/>
-							<FormControlLabel
-								control={
-									<Checkbox
-										required
-										checked={signatureCodeOfConduct}
-										onChange={(e) =>
-											setSignatureCodeOfConduct(
-												e.target.checked,
-											)
-										}
-									/>
-								}
-								label={
-									<Typography>
-										I agree with the{' '}
-										<Link
-											target="_blank"
-											href="/THCodeOfConduct.pdf"
-											className={styles.link}
-										>
-											TartanHacks Code of Conduct
-										</Link>
-										.*
-									</Typography>
-								}
-							/>
 							<FormControlLabel
 								control={
 									<Checkbox

src/components/dashboard/DashboardDialog.tsx

@@ -47,12 +47,13 @@ const getDialogText = (status: Status): ReactElement => {
 				<div className={styles.dialogText}>
 					<Typography variant="body1">
 						Non-CMU students need to complete their application by
-						11:59pm EST, January 19th.
+						11:59pm EST, January 24th. The deadline to be eligible
+						for travel reimbursement is January 3rd.
 					</Typography>
 				</div>
 				<div className={styles.dialogText}>
 					<Typography variant="body1">
-						CMU students have time until January 26th.
+						CMU students have until January 31st.
 					</Typography>
 				</div>
 			</>

src/components/form/ApplicationForm/BasicSection.tsx

@@ -245,6 +245,7 @@ const BasicSection = ({
 				error={ageErrorStatus}
 				helperText={ageHelper}
 				variant="outlined"
+				required
 				fullWidth
 				value={age}
 				onChange={(e) => {

src/components/form/ApplicationForm/TravelSection.tsx

@@ -31,10 +31,22 @@ const TravelSection = ({
 	const [wantsTravelReimbursement, setWantsTravelReimbursement] = useState<boolean>(false);
 	const [travelDetails, setTravelDetails] = useState<string>('');
 
+	// Error fields
+	const [travelInfoErrorStatus, setTravelInfoErrorStatus] = useState(false);
+	const [travelInfoHelper, setTravelInfoHelper] = useState<string>('');
+
 	const validateForm = async () => {
 		const data: TravelFields = { wantsTravelReimbursement, travelDetails };
 		await dispatch(actions.application.saveTravel(data));
-		setValid(true);
+
+		let valid = true;
+		if (wantsTravelReimbursement && travelDetails.length === 0) {
+			setTravelInfoErrorStatus(true);
+			setTravelInfoHelper('Please provide travel details if you want to apply for travel reimbursement');
+			valid = false;
+		}
+
+		setValid(valid);
 		setValidate(false);
 	};
 
@@ -81,6 +93,8 @@ const TravelSection = ({
 			<TextField
 				label="Travel details"
 				variant="outlined"
+				error={travelInfoErrorStatus}
+				helperText={travelInfoHelper}
 				fullWidth
 				multiline
 				value={travelDetails}

styles/ViewTeam.module.scss

@@ -1,6 +1,14 @@
 @import './variables.scss';
 @import './mixins.scss';
 
+.dialog {
+	width: 100%;
+	height: 100%;
+	display: flex;
+	align-items: center;
+	justify-content: center;
+}
+
 .title {
 	padding-top: 30px;
 	font-weight: 600;

styles/ViewTeams.module.scss

@@ -1,6 +1,14 @@
 @import './variables.scss';
 @import './mixins.scss';
 
+.dialog {
+	width: 100%;
+	height: 100%;
+	display: flex;
+	align-items: center;
+	justify-content: center;
+}
+
 .newTeamButton {
 	padding: 1em 5em !important;
 	font-size: large !important;