Skip to content
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.

Sign up for GitHub

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

feat: check email should be lowercase #1457

Closed
wants to merge 15 commits into from
Closed

feat: check email should be lowercase #1457

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
d9fb11c
feat: check email should be lowercase
evanyang1 Aug 4, 2023
7fa94d7
Implement React Hook Form attrs
Jul 30, 2023
fe068e0
remove unused import
Jul 30, 2023
90de98e
destructure errors from formstate, clean line spaces
Jul 30, 2023
9dbfeb9
errors onBlur and onChange
Jul 31, 2023
3d86b64
update error status and use helper text
Aug 1, 2023
c0161da
clean up
Aug 1, 2023
1d04387
add conditions for location
Aug 1, 2023
9b8da8e
validate github profile to test location validation
Aug 1, 2023
a789dc0
remove conditional attempt, replace double bang
Aug 1, 2023
08bbfce
confuse people with a nested ternerary
Aug 3, 2023
092b967
fix: Update ProjectForm.js
trillium Aug 7, 2023
ea0180d
build(deps): bump word-wrap from 1.2.3 to 1.2.5 in /client
dependabot[bot] Aug 7, 2023
beeeacc
build(deps): bump word-wrap from 1.2.3 to 1.2.5 in /backend
dependabot[bot] Aug 7, 2023
3746de8
!fix:
evanyang1 Aug 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion client/src/pages/CheckInForm.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,6 @@ const CheckInForm = (props) => {
setErrorMessage(error);
setIsLoading(false);
}
// }
};

const submitNewProfile = (userForm) => {
Expand Down Expand Up @@ -304,6 +303,11 @@ const CheckInForm = (props) => {
setErrorMessage("Please don't leave any fields blank");
ready = false;
}
if (userForm.email !== userForm.email.toLowerCase()) {
setIsError(true)
setErrorMessage("Email should be lowercase.");
ready = false;
}

const currYear = parseInt(moment().format('YYYY'));
const currMonth = parseInt(moment().format('MM'));
Expand Down Expand Up @@ -402,6 +406,8 @@ const CheckInForm = (props) => {
try {
if (!formInput.email) {
throw new Error('User email is required');
} else if (formInput.email !== formInput.email.toLowerCase()) {
throw new Error('User email must be lowercase');
}

setIsLoading(true);
Expand Down