planningalerts icon indicating copy to clipboard operation
planningalerts copied to clipboard

Remove confirmed attribute from models

Open henare opened this issue 8 years ago • 5 comments

#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.

henare avatar Apr 03 '16 06:04 henare

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!

stale[bot] avatar Aug 16 '21 05:08 stale[bot]

Still relevant.

I also notice the Alert model doesn't have a confirmed_at but only a confirmed field

mlandauer avatar Aug 16 '21 19:08 mlandauer

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!

stale[bot] avatar Feb 24 '22 01:02 stale[bot]

@mlandauer still relevant then I'm guessing?

katska avatar Feb 24 '22 02:02 katska

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!

stale[bot] avatar Aug 31 '22 00:08 stale[bot]