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.

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

Add data entry for polling station results #10

Merged
merged 8 commits into from
Apr 8, 2024
Merged

Conversation

praseodym
Copy link
Contributor

@praseodym praseodym commented Mar 27, 2024

Initial version with only voter and votes counts.

Closes #6

Bump version numbers and remove unnecessary default features for sqlx.
Initial version with only voter and votes counts.
- Change table primary key to `(polling_station_id, entry_number)`
- Add `entry_number` to path instead of request body
- Add 422 Unprocessable Content to OpenAPI spec, used for JSON
  decode errors. Also add a test for this.
lkleuver
lkleuver previously approved these changes Apr 3, 2024
Co-authored-by: Joep Schuurkes <145749778+jschuurk-kr@users.noreply.github.com>
jschuurk-kr
jschuurk-kr previously approved these changes Apr 3, 2024
@praseodym praseodym requested a review from lkleuver April 3, 2024 14:41
lkleuver
lkleuver previously approved these changes Apr 8, 2024
@praseodym praseodym dismissed stale reviews from lkleuver and jschuurk-kr via 68f8417 April 8, 2024 12:04
@praseodym praseodym enabled auto-merge (squash) April 8, 2024 12:07
@praseodym praseodym merged commit de757ec into main Apr 8, 2024
2 checks passed
@praseodym praseodym deleted the data-entry-api branch April 8, 2024 13:02
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.

Backend API en storage voor invoer bouwen
4 participants