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

Sync GEO data with IFRC-GO #38

Merged
merged 26 commits into from
May 10, 2024
Merged

Sync GEO data with IFRC-GO #38

merged 26 commits into from
May 10, 2024

Conversation

thenav56
Copy link
Member

@thenav56 thenav56 commented Apr 16, 2024

Addresses

Changes

  • Sync bbox and ifrc_go_id with IFRC-GO
  • Add github PR template

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests
  • permission checks (tests here too)
  • translations

For Country, Region and Admin1
Only update values for now. No addition
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch from 9d9db29 to 4e7dc6d Compare April 16, 2024 11:18
Geometry data is imported from IFRC-GO
- Ignore empty list for filters
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch 4 times, most recently from 48bc9c2 to 835f836 Compare April 24, 2024 09:50
- Add correct bbox for russia
- Add scripts for production mode
- Allow custom values for docker-compose environments variables
- Remove inject - TODO: Add as a management command
- N+1 fix for /
- Update config for isort
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch 5 times, most recently from 3c6e6af to b7be062 Compare April 26, 2024 08:32
- Add typing support
- Break into functions
- Tag expired alerts and filter-out for now
- Add command as replacement for /inject
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch 3 times, most recently from ea236d1 to 69b1e0b Compare April 28, 2024 09:13
- Add uwsgi package
- Fix logging issue
- Set TTL for celery task response result to 30min (existing 1day)
- Run 4 worker with max 10 task per worker lifecyle
NOTE: To avoid high redis memory usage
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch 2 times, most recently from 78cc9ac to e984fe9 Compare April 30, 2024 09:41
@thenav56 thenav56 marked this pull request as ready for review May 2, 2024 04:27
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch from ab1f8e6 to 87976e4 Compare May 2, 2024 15:05
thenav56 added 3 commits May 2, 2024 20:50
- Missing country data
- Update geo injector to skip is_deprecated & is_independent countries
- Use default pagination as `-pk`
@thenav56 thenav56 force-pushed the feature/ifrc-geo-sync branch from 067aaeb to ed6695e Compare May 10, 2024 10:55
@thenav56 thenav56 merged commit 3b94ddf into develop May 10, 2024
6 checks passed
@thenav56 thenav56 deleted the feature/ifrc-geo-sync branch May 10, 2024 12:32
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.

1 participant