Skip to content
This repository has been archived by the owner on Jan 1, 2023. It is now read-only.

Backend api improvements #87

Merged
merged 14 commits into from Dec 23, 2020
Merged

Backend api improvements #87

merged 14 commits into from Dec 23, 2020

Conversation

norkator
Copy link
Owner

@norkator norkator commented Dec 22, 2020

Draft pr...

Image loading is always copy paste'd from some other route so there is a lot of duplicate code which could be handled via one function.

Improvements

  • Create imageUtils which is responsible for handling all image loading tasks so that place is centralised. Done
  • Apply image utils for Cameras route. Done
  • Apply image utils for Faces route. Done
  • Apply image utils for History route. Done
  • Apply image utils for Site route. Done
  • Apply image utils for Training route. Done
  • Apply image utils for Wall route. Done

Doing now more overall testing.

@norkator norkator added the enhancement New feature or request label Dec 22, 2020
@norkator norkator self-assigned this Dec 22, 2020
@norkator norkator marked this pull request as ready for review December 23, 2020 18:58
@norkator norkator merged commit 045bb0f into master Dec 23, 2020
@norkator norkator deleted the backend-api-improvements branch December 23, 2020 20:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
1 participant