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

Alert fixes post revamp #982

Merged
merged 3 commits into from
Jun 8, 2022
Merged

Alert fixes post revamp #982

merged 3 commits into from
Jun 8, 2022

Conversation

Samyak2
Copy link
Contributor

@Samyak2 Samyak2 commented Jun 8, 2022

See commit messages for details

Samyak2 added 3 commits June 8, 2022 11:44
kpi_link_prefix was not passed to the macro
The default dark blue in gmail against the grey background was nearly
unreadable.

Used the same color as the buttons for now.

Also fixed the color of "Anomalies" heading in individual email.
The column name mapper wasn't being applied.

Sort columns correctly.

Remove index.

Add previous_value.
@Samyak2 Samyak2 added the ❗alerts Alert formatting, scheduling, etc. label Jun 8, 2022
@Samyak2 Samyak2 requested a review from Amatullah June 8, 2022 13:17
@gitpod-io
Copy link

gitpod-io bot commented Jun 8, 2022

@netlify
Copy link

netlify bot commented Jun 8, 2022

Deploy Preview for frontend-sb ready!

Name Link
🔨 Latest commit ea406ea
🔍 Latest deploy log https://app.netlify.com/sites/frontend-sb/deploys/62a0a15513073800086668ef
😎 Deploy Preview https://deploy-preview-982--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.

@Amatullah
Copy link
Member

LGTM

@Amatullah Amatullah merged commit 959d8a7 into develop Jun 8, 2022
@Amatullah Amatullah deleted the fix/alerts-post-revamp branch June 8, 2022 15:12
@Samyak2 Samyak2 added this to the v0.8.0 milestone Jun 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
❗alerts Alert formatting, scheduling, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants