Add database history to each admin member page

apiserver/apiserver/api/views.py

@@ -1128,9 +1128,15 @@ class HistoryViewSet(Base, List, Retrieve):
     def get_queryset(self):
         queryset = models.HistoryIndex.objects
 
-        if 'exclude_system' in self.request.query_params:
+        exclude_system = self.request.query_params.get('exclude_system', '') == 'true'
+        member_id = self.request.query_params.get('member_id', '')
+
+        if exclude_system:
             queryset = queryset.filter(is_system=False)
 
+        if member_id:
+            queryset = queryset.filter(owner_id=member_id)
+
         return queryset.order_by('-history_date')[:50]
 
 

webclient/src/Admin.js

@@ -118,7 +118,7 @@ export function AdminVetting(props) {
 }
 
 export function AdminHistory(props) {
-	const { token } = props;
+	const { token, filterMember } = props;
 	const [history, setHistory] = useState(historyCache);
 	const [excludeSystem, setExcludeSystem] = useState(excludeSystemCache);
 	const [focus, setFocus] = useState(focusCache);
@@ -131,7 +131,11 @@ export function AdminHistory(props) {
 	};
 
 	useEffect(() => {
-		const extra = excludeSystem ? '?exclude_system' : '';
+		let extra = '?exclude_system=' + excludeSystem;
+		if (filterMember) {
+			extra += '&member_id=' + filterMember;
+		}
+
 		requester('/history/'+extra, 'GET', token)
 		.then(res => {
 			setHistory(res.results);
@@ -148,6 +152,9 @@ export function AdminHistory(props) {
 			{!error ?
 				history ?
 					<>
+						<Header size='medium'>History</Header>
+						<p>Last 50 database changes:</p>
+
 						<Checkbox
 							label='Exclude System'
 							onChange={handleExcludeSystem}
@@ -160,13 +167,15 @@ export function AdminHistory(props) {
 									<Table.HeaderCell>Date</Table.HeaderCell>
 									<Table.HeaderCell>Username</Table.HeaderCell>
 									<Table.HeaderCell>Type</Table.HeaderCell>
-									<Table.HeaderCell>Owner</Table.HeaderCell>
+									{!filterMember && <Table.HeaderCell>Owner</Table.HeaderCell>}
 									<Table.HeaderCell>Object</Table.HeaderCell>
 									<Table.HeaderCell>Changed Fields</Table.HeaderCell>
 								</Table.Row>
 							</Table.Header>}
 
 							<Table.Body>
+								{!history.length && <p>None</p>}
+
 								{history.map(x =>
 									<React.Fragment key={x.id}>
 										<Table.Row>
@@ -177,7 +186,7 @@ export function AdminHistory(props) {
 											</Table.Cell>
 											<Table.Cell>{isMobile && 'User: '}{x.is_system ? 'System' : (x.history_user || 'Deleted User')}</Table.Cell>
 											<Table.Cell>{isMobile && 'Type: '}{x.history_type}</Table.Cell>
-											<Table.Cell>{isMobile && 'Owner: '}{x.owner_name}</Table.Cell>
+											{!filterMember && <Table.Cell>{isMobile && 'Owner: '}{x.owner_name}</Table.Cell>}
 											<Table.Cell>{isMobile && 'Object: '}{x.object_name}</Table.Cell>
 											<Table.Cell>{isMobile && 'Changed: '}{x.changes.map(x => x.field).join(', ')}</Table.Cell>
 										</Table.Row>
@@ -357,9 +366,7 @@ export function Admin(props) {
 			<p>All times are in Mountain time.</p>
 			<AdminUsage {...props} />
 
-
+			<p/>
-			<Header size='medium'>History</Header>
-			<p>Last 50 database changes:</p>
 
 			<AdminHistory {...props} />
 

webclient/src/Members.js

@@ -6,6 +6,7 @@ import { statusColor, isAdmin, isInstructor, BasicTable, staticUrl, requester }
 import { NotFound } from './Misc.js';
 import { AdminMemberInfo, AdminMemberPause, AdminMemberForm, AdminMemberCards, AdminMemberTraining, AdminMemberCertifications } from './AdminMembers.js';
 import { AdminMemberTransactions } from './AdminTransactions.js';
+import { AdminHistory } from './Admin.js';
 import { StorageButton } from './Storage.js';
 import AbortController from 'abort-controller';
 
@@ -341,7 +342,8 @@ export function MemberDetail(props) {
 								<Link to={'/members/'+member.id+'/cards'}>Cards</Link>{' - '}
 								<Link to={'/members/'+member.id+'/lockouts'}>Lockouts</Link>{' - '}
 								<Link to={'/members/'+member.id+'/training'}>Training</Link>{' - '}
-								<Link to={'/members/'+member.id+'/transactions'}>Transactions</Link>
+								<Link to={'/members/'+member.id+'/transactions'}>Transactions</Link>{' - '}
+								<Link to={'/members/'+member.id+'/history'}>History</Link>
 							</p>
 						}
 
@@ -365,27 +367,23 @@ export function MemberDetail(props) {
 							</Route>}
 
 							{isAdmin(user) && <Route path='/members/:id/cards'>
-								<Segment padded>
 								<AdminMemberCards result={result} refreshResult={refreshResult} {...props} />
-								</Segment>
 							</Route>}
 
 							{isAdmin(user) && <Route path='/members/:id/lockouts'>
-								<Segment padded>
 								<AdminMemberCertifications result={result} refreshResult={refreshResult} {...props} />
-								</Segment>
 							</Route>}
 
 							{isAdmin(user) && <Route path='/members/:id/training'>
-								<Segment padded>
 								<AdminMemberTraining result={result} refreshResult={refreshResult} {...props} />
-								</Segment>
 							</Route>}
 
 							{isAdmin(user) && <Route path='/members/:id/transactions'>
-								<Segment padded>
 								<AdminMemberTransactions result={result} refreshResult={refreshResult} {...props} />
-								</Segment>
+							</Route>}
+
+							{isAdmin(user) && <Route path='/members/:id/history'>
+								<AdminHistory filterMember={member.id} {...props} />
 							</Route>}
 
 							<Route path='/members/:id'>