Skip to content

Feature/save eligible voters list#129

Merged
perkynades merged 17 commits intodevelopfrom
feature/save-eligible-voters-list
Mar 11, 2021
Merged

Feature/save eligible voters list#129
perkynades merged 17 commits intodevelopfrom
feature/save-eligible-voters-list

Conversation

@perkynades
Copy link
Copy Markdown
Contributor

Closes #121

This pull request is closely related to pull request nr. 115 at frontend

Comment thread src/helpers/validateEntity.ts Outdated
Comment on lines +46 to +49
// @MinDate(new Date(), {
// groups: ['creation'],
// always: false
// })
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this commented out?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It throws error, even when the dates are set correctly. I have written an issue on it

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I fix

Comment thread src/services/EligibleVoterService.ts Outdated
Comment thread src/services/EligibleVoterService.ts
@perkynades
Copy link
Copy Markdown
Contributor Author

The tests failing are due to a bug in the date constraints of "ElectionEntity", i have commented these out to not get an error in compilation. But as i have commented these out, some of the tests fail.

@perkynades perkynades requested a review from freshfish70 March 10, 2021 16:33
Comment thread src/services/EligibleVoterService.ts
Comment thread tests/services/EligibleVoterService.test.ts Outdated
Comment thread src/helpers/email.ts
@spiftire spiftire added this to the Sprint 9 milestone Mar 11, 2021
@perkynades perkynades merged commit 28c57c2 into develop Mar 11, 2021
@perkynades perkynades deleted the feature/save-eligible-voters-list branch March 11, 2021 15:12
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.

Save list of eligible voters to the database

3 participants