Log event creation to admin audit log, add back button, bump to 1.10.3
createEvent never called logAdminAction, even though 1.10.0 already listed event_created as a filterable action on the audit-log page — only updateEvent/deleteEvent actually logged. Event creation is now logged the same way, at every return path including the legacy pre-migration retry branches. Also adds a "Back to dashboard" link to Admin -> Audit log, matching the existing back-link pattern on the Cashup page. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CSWFWQsjTc9GyffPiXEDQT
This commit is contained in:
@@ -7,6 +7,13 @@ and this project follows [Semantic Versioning](https://semver.org/).
|
|||||||
|
|
||||||
## [Unreleased]
|
## [Unreleased]
|
||||||
|
|
||||||
|
## [1.10.3] - 2026-08-28
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
|
||||||
|
- Creating an event was never recorded in the admin audit log — 1.10.0 added `event_created` as a logged action on the frontend's filter list, but the backend's `createEvent` never actually called `logAdminAction`, only `updateEvent`/`deleteEvent` did. Event creation is now logged the same way.
|
||||||
|
- Added a "Back to dashboard" link to Admin → Audit log, matching the back-link pattern already used on the Cashup page.
|
||||||
|
|
||||||
## [1.10.2] - 2026-08-28
|
## [1.10.2] - 2026-08-28
|
||||||
|
|
||||||
### Fixed
|
### Fixed
|
||||||
|
|||||||
Generated
+2
-2
@@ -1,12 +1,12 @@
|
|||||||
{
|
{
|
||||||
"name": "event-management-backend",
|
"name": "event-management-backend",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "event-management-backend",
|
"name": "event-management-backend",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"hasInstallScript": true,
|
"hasInstallScript": true,
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@prisma/client": "^5.4.2",
|
"@prisma/client": "^5.4.2",
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "event-management-backend",
|
"name": "event-management-backend",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"description": "Event Management System Backend",
|
"description": "Event Management System Backend",
|
||||||
"main": "src/index.js",
|
"main": "src/index.js",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
|
|||||||
@@ -68,8 +68,19 @@ const createEvent = async (req, res) => {
|
|||||||
location: location || null,
|
location: location || null,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const logEventCreate = (createdEvent) => logAdminAction({
|
||||||
|
actorId: req.user?.id,
|
||||||
|
actorRole: req.user?.role,
|
||||||
|
action: 'event_created',
|
||||||
|
targetType: 'Event',
|
||||||
|
targetId: createdEvent.id,
|
||||||
|
metadata: { title: createdEvent.title },
|
||||||
|
ip: getClientIp(req),
|
||||||
|
});
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const event = await prisma.event.create({ data });
|
const event = await prisma.event.create({ data });
|
||||||
|
logEventCreate(event);
|
||||||
|
|
||||||
// Automatically create a main ticket (event option) with the event price — contact-only
|
// Automatically create a main ticket (event option) with the event price — contact-only
|
||||||
// events have no bookable options, so there's nothing to auto-create for them.
|
// events have no bookable options, so there's nothing to auto-create for them.
|
||||||
@@ -133,24 +144,28 @@ const createEvent = async (req, res) => {
|
|||||||
// @ts-ignore delete field and retry
|
// @ts-ignore delete field and retry
|
||||||
delete data.registrationDeadline;
|
delete data.registrationDeadline;
|
||||||
const event = await prisma.event.create({ data });
|
const event = await prisma.event.create({ data });
|
||||||
|
logEventCreate(event);
|
||||||
return res.status(201).json(event);
|
return res.status(201).json(event);
|
||||||
}
|
}
|
||||||
if (msg.includes('Unknown argument `goLiveAt`')) {
|
if (msg.includes('Unknown argument `goLiveAt`')) {
|
||||||
// @ts-ignore delete field and retry
|
// @ts-ignore delete field and retry
|
||||||
delete data.goLiveAt;
|
delete data.goLiveAt;
|
||||||
const event = await prisma.event.create({ data });
|
const event = await prisma.event.create({ data });
|
||||||
|
logEventCreate(event);
|
||||||
return res.status(201).json(event);
|
return res.status(201).json(event);
|
||||||
}
|
}
|
||||||
if (msg.includes('Unknown argument `createdById`')) {
|
if (msg.includes('Unknown argument `createdById`')) {
|
||||||
// @ts-ignore delete field and retry
|
// @ts-ignore delete field and retry
|
||||||
delete data.createdById;
|
delete data.createdById;
|
||||||
const event = await prisma.event.create({ data });
|
const event = await prisma.event.create({ data });
|
||||||
|
logEventCreate(event);
|
||||||
return res.status(201).json(event);
|
return res.status(201).json(event);
|
||||||
}
|
}
|
||||||
if (msg.includes('Unknown argument `redirectUrl`')) {
|
if (msg.includes('Unknown argument `redirectUrl`')) {
|
||||||
// @ts-ignore delete field and retry
|
// @ts-ignore delete field and retry
|
||||||
delete data.redirectUrl;
|
delete data.redirectUrl;
|
||||||
const event = await prisma.event.create({ data });
|
const event = await prisma.event.create({ data });
|
||||||
|
logEventCreate(event);
|
||||||
return res.status(201).json(event);
|
return res.status(201).json(event);
|
||||||
}
|
}
|
||||||
throw err;
|
throw err;
|
||||||
|
|||||||
Generated
+2
-2
@@ -1,12 +1,12 @@
|
|||||||
{
|
{
|
||||||
"name": "hope-events-frontend",
|
"name": "hope-events-frontend",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "hope-events-frontend",
|
"name": "hope-events-frontend",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"@hookform/resolvers": "^5.2.1",
|
"@hookform/resolvers": "^5.2.1",
|
||||||
"@radix-ui/react-accordion": "^1.2.11",
|
"@radix-ui/react-accordion": "^1.2.11",
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "hope-events-frontend",
|
"name": "hope-events-frontend",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"private": true,
|
"private": true,
|
||||||
"scripts": {
|
"scripts": {
|
||||||
"dev": "next dev --turbopack",
|
"dev": "next dev --turbopack",
|
||||||
|
|||||||
@@ -113,7 +113,8 @@ export default function AdminAuditLogPage() {
|
|||||||
<History className="w-5 h-5 text-brand-600" />
|
<History className="w-5 h-5 text-brand-600" />
|
||||||
</div>
|
</div>
|
||||||
<div>
|
<div>
|
||||||
<h1 className="text-2xl font-semibold text-gray-900">Admin Audit Log</h1>
|
<button className="text-xs text-brand-600 hover:underline" onClick={() => router.push("/dashboard/admin")}>← Back to dashboard</button>
|
||||||
|
<h1 className="text-2xl font-semibold text-gray-900 mt-0.5">Admin Audit Log</h1>
|
||||||
<p className="text-sm text-gray-500">{total} action{total !== 1 ? "s" : ""} recorded</p>
|
<p className="text-sm text-gray-500">{total} action{total !== 1 ? "s" : ""} recorded</p>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Generated
+2
-2
@@ -1,12 +1,12 @@
|
|||||||
{
|
{
|
||||||
"name": "hope-events",
|
"name": "hope-events",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "hope-events",
|
"name": "hope-events",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"license": "ISC",
|
"license": "ISC",
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"concurrently": "^9.2.1"
|
"concurrently": "^9.2.1"
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "hope-events",
|
"name": "hope-events",
|
||||||
"version": "1.10.2",
|
"version": "1.10.3",
|
||||||
"main": "index.js",
|
"main": "index.js",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
"dev:backend": "cd backend && npm run dev",
|
"dev:backend": "cd backend && npm run dev",
|
||||||
|
|||||||
Reference in New Issue
Block a user