Skip to content

Fix/fr queries#681

Merged
GergiH merged 2 commits intodevelopfrom
fix/fr-queries
Apr 27, 2020
Merged

Fix/fr queries#681
GergiH merged 2 commits intodevelopfrom
fix/fr-queries

Conversation

@batpad
Copy link
Copy Markdown
Collaborator

@batpad batpad commented Apr 24, 2020

Addresses IFRCGo/go-frontend#1072 (comment)

Changes

  • Optimizes SQL queries on field report API end-point, reducing no of queries from 300+ to 9.

Checklist

Things that should succeed before merging.

  • Field Report API endpoint should work same as before, just much faster.

cc @GregoryHorvath

@batpad batpad changed the base branch from master to develop April 24, 2020 16:37
@batpad
Copy link
Copy Markdown
Collaborator Author

batpad commented Apr 27, 2020

@GregoryHorvath this should be good to merge to staging and test 👍

@GergiH GergiH merged commit 3045af3 into develop Apr 27, 2020
@thenav56 thenav56 deleted the fix/fr-queries branch June 2, 2020 12:28
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.

2 participants