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

Routes poll #46

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

medro25
Copy link

@medro25 medro25 commented Mar 5, 2024

Please , Could you possibly review it ?

jkerola and others added 22 commits January 8, 2024 14:56
* add workflows

* add pre-commit hooks

* update readme

* format meetings.md
* add minimal server, container setup

* fix prettier config

* fix workflow errors

* remove setup env workflow due to monorepo structure
* add basic auth setup

* fix unit tests

* fix tsconfig.json formatting
update pre-commit config
Update group info.
* update pre-commit, secrets and gitignore

* delete nestjs, add venv template

* update requests

* update readme

* update github actions for python

* update workflow params, fix requirements.txt path

* add custom setup-env action for python

* fix requirements.txt path again
* update requirements.txt

* add prisma schema

* add prisma initialization

* add containerization files

* update readme

* update api readme

* add adminer, fix readme contents

* update gitignore

* update readme

* update dockerignore

* disable prod image, use asyncio for db connection

* capitalize docker commands
Add meeting 1 notes
* add auth setup

* rework error handling

* improve error handling

* move expires_in to config

* fix assignment on previous commit

* improve jwt error handling

* update requirements.txt

* format meetings.md

* update readme

* add chart title

* add comments and rename functions
* abstract funcs to BaseDto

* add sql dump, instructions
Meeting 1 checkboxes
* Meeting 2 notes

* formatted meetings.md

---------

Co-authored-by: Emilia Pyyny <[email protected]>
Co-authored-by: Janne Kerola <[email protected]>
* add poll route

Remove rebase error

Fix more errors

Remove extra space

remove debug prints

Rename import

* fix jannes mistakes

* add poll, pollitem

* Add converters

* Update controllers

* Update requests

* Add converters to a separate dir

* Add pollitem update, delete

* fix route names, linter errors

* fix formatting errors

---------

Co-authored-by: Emilia Pyyny <[email protected]>
Co-authored-by: Janne Kerola <[email protected]>
* Add docstrings

* format files

---------

Co-authored-by: Emilia Pyyny <[email protected]>
Co-authored-by: Janne Kerola <[email protected]>
* fix middleware

* fix converters

* fix auth.py

* fix models

* fix services

* fix rest
* add auth route tests

* update gitignore

* improve readability

* update setup to run only once
* protect POST poll route

* refactor poll routes

* add vote func

* fix pylint warnings
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.

4 participants