Skip to content
This repository was archived by the owner on Sep 12, 2024. It is now read-only.

fix(migrations): ignore missing percent_change #1144

Merged
merged 1 commit into from
Sep 30, 2022

Conversation

Samyak2
Copy link
Contributor

@Samyak2 Samyak2 commented Sep 30, 2022

Some very old triggered alerts don't have percent_change.

Some very old triggered alerts don't have percent_change.
@Samyak2 Samyak2 added 🛠️ backend ❗alerts Alert formatting, scheduling, etc. ⚠️ has db migration The changes contain a new database migration. Use caution before merging. labels Sep 30, 2022
@gitpod-io
Copy link

gitpod-io bot commented Sep 30, 2022

@netlify
Copy link

netlify bot commented Sep 30, 2022

Deploy Preview for frontend-sb ready!

Name Link
🔨 Latest commit a8fb6a9
🔍 Latest deploy log https://app.netlify.com/sites/frontend-sb/deploys/6336949b5a18be00085e2980
😎 Deploy Preview https://deploy-preview-1144--frontend-sb.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@manassolanki manassolanki merged commit f30b6ee into develop Sep 30, 2022
@Samyak2 Samyak2 deleted the fix/migration-ignore-missing-percent-change branch September 30, 2022 07:12
@Samyak2 Samyak2 added this to the v0.11.0 milestone Oct 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
❗alerts Alert formatting, scheduling, etc. 🛠️ backend ⚠️ has db migration The changes contain a new database migration. Use caution before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants