-
-
Notifications
You must be signed in to change notification settings - Fork 958
Add users.policies_acknowledged_at #5530
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add users.policies_acknowledged_at #5530
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5530 +/- ##
=======================================
Coverage 97.15% 97.15%
=======================================
Files 460 460
Lines 9552 9559 +7
=======================================
+ Hits 9280 9287 +7
Misses 272 272 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
1a7bc4d
to
e7114fa
Compare
e7114fa
to
b08b505
Compare
So we can track when users last acked the policies, and prompt them to do so when we introduce/update them Signed-off-by: Samuel Giddins <segiddins@segiddins.me>
b08b505
to
5e23c83
Compare
@mghaught would you mind to update what's the last take on this? How this approval is going to be collected? |
@simi as per the email:
I'm going to add this functionality into a separate PR |
So we can track when users last acked the policies, and prompt them to do so when we introduce/update them
Signed-off-by: Samuel Giddins segiddins@segiddins.me