Project/phase 2 #95
Merged
Project/phase 2 #95
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thenav56
force-pushed
the
project/phase-2
branch
3 times, most recently
from
6d962f6 to
f827afc
Compare
thenav56
force-pushed
the
project/phase-2
branch
2 times, most recently
from
b1e8a61 to
fef661d
Compare
- Update pyright version - Add collectstatic for run_web.sh
* Add subscription models/query/mutations * Monkey patch weird init error - Upgrade strawberry-django: 0.47.1 -> 0.48.1 * Restructure subscription attributes * Add subscription alert tagging tasks * Break Alert.is_processed_by_subscription field into 2 migrations To avoid full rewrite of the table https://docs.djangoproject.com/en/5.1/ref/migration-operations/#addfield * Change subscription is_active default as False - Add test cases for subscription limit per user * Subscription email basic setup - Add unsubscribe_url - Fix subscription filtering typing issue
- Change filter structure to use index - Fix ID-name issues for filterAlertCountry - Add historical alerts node
* Add argocd hooks for migraitons and collectstatic * Fix argoCD hook issues - Lower docker file size
* Update subscriptions email template - Add helper script to create fake alerts - Remove legacy templates - Add helper dev views to update templates * Fix email templates * Fix email raw text templates - Truncate alert admins to 40 chars * Add error catch - Update TODOs - Upgrade pyright --------- Co-authored-by: thenav56 <ayernavin@gmail.com>
6 tasks
* Add Admin1 geo code to database - Add sync logic to pull data from go api (currently stage) * Add tagging from circle and geocodes as fallback * User _value2member_map_ to check geocode type is supported by using enum * Fix mapping value issue * Use go-api prod for geo data sync * Skip circle polygon filter if tagged by normal polygons * Use camelCase for argo hooks * Add support for custom python script in argoHooks * Add missing cronjobs for WEEKLY and MONTHLY subscription email * Add total_alerts_count node for subscription alerts count - Fix filtering issue for subscription alerts * Disable querystring_auth for S3 static bucket * Add test additional dataset for subscription/alert tagging test * Add multiple queue: feeds and default * Add celery flower * Fix read me and email template * Typos fix --------- Co-authored-by: barshathakuri <barshathakuri708@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addresses
Changes
Post commands
This PR doesn't introduce any:
printThis PR contains valid: