Hi there! I've opened a PR after upgrading to `0.1...
# community-support
s
Hi there! I've opened a PR after upgrading to
0.14.1
version. Seems like minor macro change for type cast. Could anyone support with that PR review ? https://github.com/elementary-data/elementary/pull/1451
✅ 1
i
Hey @Serhii Dimchenko, thank you for raising that up and for the contribution! I commented on the PR itself, and will share it here as well 🙂 The reason why
sent_at
was set to be a string is because it was a string on the older version of the alerts and the alerts_v2 was populated from it. I am not against this change, specially when this is breaking things for you, but we need to make sure it doesn't break other adapters or users that already have alerts_v2 table with sent_at as string. There is a way to do it, and it should not be hard, I will provide more guidance in a bit 🙂
I commented the guidance on the PR how to make sure we don't break things for other users 🙂 Feel free to reach out any time for any question or more guidance, and once again thank you for the contribution!