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.

Sign up for GitHub

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

Jump to bottom

Refactor backend poi endpoint. #287

Closed
JoJoensuu opened this issue Aug 20, 2023 · 0 comments · Fixed by #288
Closed

Refactor backend poi endpoint. #287

JoJoensuu opened this issue Aug 20, 2023 · 0 comments · Fixed by #288
Assignees
@JoJoensuu
Labels
task Improvements or additions to documentation
Milestone
Sprint 5

Comments

@JoJoensuu
Copy link
Contributor

Too many responsibilities in single functions.
Testing manager.py seems impossible.
Poi.py doing too many things as well.
@JoJoensuu JoJoensuu added the task Improvements or additions to documentation label Aug 20, 2023
@JoJoensuu JoJoensuu added this to the Sprint 5 milestone Aug 20, 2023
@JoJoensuu JoJoensuu self-assigned this Aug 20, 2023
@JoJoensuu JoJoensuu mentioned this issue Aug 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JoJoensuu JoJoensuu

Labels
task Improvements or additions to documentation
Projects
Project Backlog
Status: Approved by customer
Milestone
Sprint 5
Development

Successfully merging a pull request may close this issue.

moved data fetching HelsinkiUniCollab/WeatherBasedRecommender
1 participant
@JoJoensuu