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

Fix/readme #109

Closed
wants to merge 16 commits into from
Closed

Fix/readme #109

wants to merge 16 commits into from

Conversation

barshathakuri
Copy link
Contributor

Changes

  • Update readme file
  • Fix email 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

thenav56 and others added 16 commits November 27, 2024 17:15
- 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 <[email protected]>
@barshathakuri barshathakuri changed the base branch from develop to project/phase-2 December 6, 2024 09:40
@barshathakuri barshathakuri changed the base branch from project/phase-2 to feat/admin1-geo-code December 6, 2024 09:41
@barshathakuri barshathakuri changed the base branch from feat/admin1-geo-code to project/phase-2 December 6, 2024 09:41
@thenav56 thenav56 changed the base branch from project/phase-2 to feat/admin1-geo-code December 8, 2024 12:24
@thenav56
Copy link
Member

thenav56 commented Dec 8, 2024

Included in #106

@thenav56 thenav56 closed this Dec 8, 2024
@thenav56 thenav56 deleted the fix/readme branch December 8, 2024 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.

3 participants