-
Notifications
You must be signed in to change notification settings - Fork 3
feat: teams #102
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
base: main
Are you sure you want to change the base?
feat: teams #102
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #102 +/- ##
===========================================
- Coverage 82.70% 72.22% -10.49%
===========================================
Files 47 5 -42
Lines 1376 108 -1268
Branches 168 25 -143
===========================================
- Hits 1138 78 -1060
+ Misses 238 30 -208
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 50 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
# - name: test | ||
# uses: borales/[email protected] | ||
# with: | ||
# cmd: backend::test |
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.
are tests really failing
* Get all users | ||
* @returns all users | ||
*/ | ||
public async getAllUsers(): Promise<readonly UserListDto[]> { |
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.
extend comment
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.
getSemiAnyomouseUsers()
/** | ||
* Get all users | ||
*/ | ||
public async getAllUsers(): Promise<UserListDto[]> { |
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.
getSemiAnyomouseUsers()
/** | ||
* Get user list only with names and ids | ||
*/ | ||
@Get("/list") |
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.
getSemiAnyomouseUsers()
/> | ||
<div style={{ marginTop: "1rem" }}> | ||
<MuiButton | ||
href="whatsapp://send?text=I am visiting Hackaburg this year! Join me at https://hackaburg.de" |
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.
whatsapp invite raus
to: string; | ||
children: string; | ||
children: any; | ||
} |
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.
what is children?
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.
correct type or comment
import { Page } from "./page"; | ||
import { saveAs } from "file-saver"; | ||
import { Divider } from "../base/divider"; | ||
import { BsGenderFemale, BsGenderMale } from "react-icons/bs"; |
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.
show also non-binary gender in application form
<TableCell onClick={handleExpandRow}>{email}</TableCell> | ||
<TableCell onClick={handleExpandRow}>{name}</TableCell> | ||
<TableCell onClick={handleExpandRow}>{teamNumber}</TableCell> | ||
<TableCell onClick={handleExpandRow}> |
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.
extend
RowComponent = AdmittedRow; | ||
} | ||
|
||
const cityIndex = questions.find((q) => q.title === "City")?.id!; |
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.
city / country / gender is mandatory in settings => add comment in subheadline
No description provided.