Skip to content

Commit

Permalink
feat(client): expanded display and access of audit log page to all au…
Browse files Browse the repository at this point in the history
…thenticated users

affects: client

ISSUES CLOSED: #391
  • Loading branch information
grioux-neo authored and r-a303931 committed Nov 30, 2023
1 parent 5810428 commit 7aecf35
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 11 deletions.
11 changes: 1 addition & 10 deletions packages/client/src/pages/events/AuditViewer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,13 @@ import {
AuditableEvents,
ChangeEvent,
DeleteEvent,
effectiveManageEventPermissionForEvent,
Either,
EventAuditEvents,
EventObject,
formatEventViewerDate as formatDate,
FromDatabase,
getURIComponent,
NewEventObject,
Permissions,
} from 'common-lib';
import * as React from 'react';
import DropDownList from '../../components/DropDownList';
Expand Down Expand Up @@ -88,7 +86,7 @@ export default class AuditViewer extends Page<AuditViewerProps> {
]);

this.props.deleteReduxState();

if (Either.isLeft(informationEither)) {
this.setState(prev => ({
...prev,
Expand Down Expand Up @@ -149,13 +147,6 @@ export default class AuditViewer extends Page<AuditViewerProps> {
const eventAuditInfo = this.state.auditInformation;
const eventInfo = this.state.eventInformation;

if (
effectiveManageEventPermissionForEvent(this.props.member)(eventInfo) !==
Permissions.ManageEvent.FULL
) {
return <div>This page requires management permissions to view.</div>;
}

return (
<>
<div className="auditviewerroot">
Expand Down
4 changes: 3 additions & 1 deletion packages/client/src/pages/events/EventViewer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,7 @@ export class EventViewer extends Page<EventViewerProps, EventViewerState> {
);

this.props.deleteReduxState();

if (Either.isLeft(eventInformation)) {
this.setState(prev => ({
...prev,
Expand Down Expand Up @@ -710,6 +710,8 @@ export class EventViewer extends Page<EventViewerProps, EventViewerState> {
{' | '}
<Link to={`/multiadd/${event.id}`}>Add attendance</Link>
</>
) : member ? (
<Link to={`/auditviewer/${event.id}`}>View Audit Log</Link>
) : null}
{(member && effectiveManageEventPermissionForEvent(member)(event)) ||
(fullMemberDetails.error === MemberCreateError.NONE &&
Expand Down

0 comments on commit 7aecf35

Please sign in to comment.