planningalerts
planningalerts copied to clipboard
Remove confirmed attribute from models
#931 introduced confirmed_at
on the Comment model so confirmed
is no longer required - we can just check if confirmed_at
is present instead. We should be able to do the same for Alert and reduce this duplication.
This issue has been automatically marked as stale because there has been no activity on it for a year. If you want to keep it open please make a comment and explain why this issue is still relevant. Otherwise it will be automatically closed in a week. Thank you!
Still relevant.
I also notice the Alert
model doesn't have a confirmed_at
but only a confirmed
field
This issue has been automatically marked as stale because there has been no activity on it for about six months. If you want to keep it open please make a comment and explain why this issue is still relevant. Otherwise it will be automatically closed in a week. Thank you!
@mlandauer still relevant then I'm guessing?
This issue has been automatically marked as stale because there has been no activity on it for about six months. If you want to keep it open please make a comment and explain why this issue is still relevant. Otherwise it will be automatically closed in a week. Thank you!