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

Jump to bottom

AgentEinstein/add error checking #863

Merged
merged 6 commits into from
Nov 13, 2023
Merged

AgentEinstein/add error checking #863

Changes from 4 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
8ee64a0
Merge branch 'dev' of https://github.com/acm-ucr/hackathon-website in…
AgentEinstein Oct 24, 2023
fa1cd7c
Merge branch 'dev' of https://github.com/acm-ucr/hackathon-website in…
AgentEinstein Nov 5, 2023
833af73
Added Empty Team ID Error Checking
AgentEinstein Nov 5, 2023
282a6aa
formatting
AgentEinstein Nov 5, 2023
3c90448
slight logic change (functionality unchanged)
AgentEinstein Nov 5, 2023
c2588d0
fix
shahdivyank Nov 13, 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
4 changes: 4 additions & 0 deletions src/components/dynamic/user/Team.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,10 @@ const Team = ({ user, setUser }) => {
};

const handleJoin = () => {
if (id.team.length === 0) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

u can just check for an empty string instead of character comparison

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

toast("❌ Enter a Valid Team ID");
return;
}
axios
.put("/api/members", { team: id.team })
.then(() => {
Expand Down