From 3ec76e4cfd95586c0d67112b733eea5bcb68b586 Mon Sep 17 00:00:00 2001
From: Tanner Collin <git@tannercollin.com>
Date: Fri, 30 Dec 2022 23:52:11 +0000
Subject: [PATCH] Automatically un-vet members away for more than a year

---
 apiserver/apiserver/api/views.py | 21 ++++++++++++++-
 webclient/src/AdminMembers.js    | 46 ++++++++++++++++++++++++++------
 2 files changed, 58 insertions(+), 9 deletions(-)

diff --git a/apiserver/apiserver/api/views.py b/apiserver/apiserver/api/views.py
index 9ff500f..d323df9 100644
--- a/apiserver/apiserver/api/views.py
+++ b/apiserver/apiserver/api/views.py
@@ -196,11 +196,29 @@ class MemberViewSet(Base, Retrieve, Update):
     def unpause(self, request, pk=None):
         if not is_admin_director(self.request.user):
             raise exceptions.PermissionDenied()
+
+        today = utils.today_alberta_tz()
         member = self.get_object()
-        member.current_start_date = utils.today_alberta_tz()
+
+        difference = utils.today_alberta_tz() - member.paused_date
+        if difference.days > 370:  # give some leeway
+            logging.info('Member has been away for %s days (since %s), unvetting...', difference.days, member.paused_date)
+            member.vetted_date = None
+            member.orientation_date = None
+            member.lathe_cert_date = None
+            member.mill_cert_date = None
+            member.wood_cert_date = None
+            member.wood2_cert_date = None
+            member.tormach_cnc_cert_date = None
+            member.precix_cnc_cert_date = None
+            member.rabbit_cert_date = None
+            member.trotec_cert_date = None
+
+        member.current_start_date = today
         member.paused_date = None
         if not member.monthly_fees:
             member.monthly_fees = 55
+
         member.save()
         utils.tally_membership_months(member)
         utils.gen_member_forms(member)
@@ -560,6 +578,7 @@ class DoorViewSet(viewsets.ViewSet, List):
         for card in cards:
             member = card.user.member
             if member.paused_date: continue
+            if not member.vetted_date: continue
             if not member.is_allowed_entry: continue
 
             active_member_cards[card.card_number] = '{} ({})'.format(
diff --git a/webclient/src/AdminMembers.js b/webclient/src/AdminMembers.js
index 8c6addf..37f1670 100644
--- a/webclient/src/AdminMembers.js
+++ b/webclient/src/AdminMembers.js
@@ -124,7 +124,7 @@ let prevAutoscan = '';
 export function AdminMemberCards(props) {
 	const { token, result, refreshResult } = props;
 	const cards = result.cards;
-	const startDimmed = Boolean((result.member.paused_date || !result.member.is_allowed_entry) && cards.length);
+	const startDimmed = Boolean((result.member.paused_date || !result.member.is_allowed_entry || !result.member.vetted_date) && cards.length);
 	const [dimmed, setDimmed] = useState(startDimmed);
 	const [input, setInput] = useState({ active_status: 'card_active' });
 	const [error, setError] = useState(false);
@@ -134,7 +134,7 @@ export function AdminMemberCards(props) {
 	const { id } = useParams();
 
 	useEffect(() => {
-		const startDimmed = Boolean((result.member.paused_date || !result.member.is_allowed_entry) && cards.length);
+		const startDimmed = Boolean((result.member.paused_date || !result.member.is_allowed_entry || !result.member.vetted_date) && cards.length);
 		setDimmed(startDimmed);
 	}, [result.member]);
 
@@ -298,7 +298,7 @@ export function AdminMemberCards(props) {
 
 				<Dimmer active={dimmed}>
 					<p>
-						Member paused or not allowed entry, {cards.length} card{cards.length === 1 ? '' : 's'} ignored anyway.
+						Member paused, unvetted or not allowed entry. {cards.length} card{cards.length === 1 ? '' : 's'} ignored anyway.
 					</p>
 					<p>
 						<Button size='tiny' onClick={() => setDimmed(false)}>Close</Button>
@@ -363,15 +363,45 @@ export function AdminMemberPause(props) {
 		<div>
 			<Header size='medium'>Pause / Unpause Membership</Header>
 
-			<p>Pause members who are inactive, former, or on vacation.</p>
-
 			<p>
 				{result.member.paused_date ?
-					<Button onClick={handleUnpause} loading={loading}>
-						Unpause
-					</Button>
+					result.member.vetted_date && moment().diff(moment(result.member.paused_date), 'days') > 370 ?
+						<>
+							<p>
+								{result.member.preferred_name} has been away for more than a year and will need to be re-vetted according to our
+								<a href='https://wiki.protospace.ca/Approved_policies/Membership' target='_blank' rel='noopener noreferrer'> policy</a>.
+							</p>
+							<p>
+								<Form.Checkbox
+									name='told_subscriptions'
+									value={told1}
+									label='Told member to get re-vetted'
+									required
+									onChange={(e, v) => setTold1(v.checked)}
+								/>
+							</p>
+							<p>
+								<Form.Checkbox
+									name='told_shelves'
+									value={told2}
+									label='Collected payment for member dues'
+									required
+									onChange={(e, v) => setTold2(v.checked)}
+								/>
+							</p>
+
+							<Button onClick={handleUnpause} loading={loading} disabled={!told1 || !told2}>
+								Unpause
+							</Button>
+						</>
+					:
+						<Button onClick={handleUnpause} loading={loading}>
+							Unpause
+						</Button>
 				:
 					<>
+						<p>Pause members who are inactive, former, or on vacation.</p>
+
 						<p>
 							<Form.Checkbox
 								name='told_subscriptions'