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

Remove broken Cosmos gov notification generation #8852

Merged
merged 3 commits into from
Aug 16, 2024

Conversation

timolegros
Copy link
Collaborator

Link to Issue

Closes: #8851
Closes as not planned: #6709
Closes as not planned: #7672
Related to: #7738
Related to: #8156

Description of Changes

  • Removed Cosmos governance notification generator

Test Plan

Ensure app starts like normal

Deployment Plan

  1. Remove the Heroku scheduler job for the Cosmos notification generator

Other Considerations

Copy link
Contributor

@ianrowan ianrowan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jnaviask jnaviask merged commit e75f1be into master Aug 16, 2024
9 checks passed
@timolegros timolegros added the deployment plan (PRs only) requires manual infrastructure changes on release label Aug 19, 2024
@timolegros timolegros mentioned this pull request Aug 19, 2024
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment plan (PRs only) requires manual infrastructure changes on release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Cosmos Gov Notifications
3 participants