Skip to content

Feature/add election organizer to election#103

Merged
spiftire merged 9 commits intodevelopfrom
feature/add-election-organizer-to-election
Feb 24, 2021
Merged

Feature/add election organizer to election#103
spiftire merged 9 commits intodevelopfrom
feature/add-election-organizer-to-election

Conversation

@perkynades
Copy link
Copy Markdown
Contributor

Closes #88

Comment thread src/services/ElectionOrganizerService.ts Outdated
Comment thread src/api/routes/elections/index.ts Outdated
Comment on lines 19 to 23
electionDTO.electionOrganizer = request.electionOrganizer

if (!electionDTO || isObjectEmpty(electionDTO)) {
throw new BadRequestError({ message: 'Empty request' })
}
Copy link
Copy Markdown
Contributor

@sanderhurlen sanderhurlen Feb 24, 2021

Choose a reason for hiding this comment

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

⚠️ ⚠️ This would make the request never empty!!

@spiftire spiftire merged commit 13eef58 into develop Feb 24, 2021
@spiftire spiftire deleted the feature/add-election-organizer-to-election branch February 24, 2021 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When creating an election, the logged in election organizer needs to be added to the eleciton object

4 participants