Remove AccentGraphic

Author: o-seliuchenko

src/controllers/bannerController.ts

@@ -408,8 +408,6 @@ function extractFileUrls(banner: any): string[] {
   if (banner.Logo?.Url) urls.push(banner.Logo.Url);
   if (banner.BackgroundImage?.Url) urls.push(banner.BackgroundImage.Url);
   if (banner.MainImage?.Url) urls.push(banner.MainImage.Url);
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // if (banner.AccentGraphic?.Url) urls.push(banner.AccentGraphic.Url);
 
   // Partner logos for partnership charter banners (nested structure)
   if (banner.PartnershipCharter?.PartnerLogos && Array.isArray(banner.PartnershipCharter.PartnerLogos)) {
@@ -432,8 +430,6 @@ function processMediaFields(req: Request): any {
   // Note: I still not sure if it makes sense to use this merging instead of req.body
   const processedData = { ...req.body, ...req.preValidatedData };
 
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // Process single media assets: Logo, BackgroundImage, MainImage, AccentGraphic
   ['Logo', 'BackgroundImage', 'MainImage' /*, 'AccentGraphic'*/].forEach(field => {
     const newFileData = processedData[`newfile_${field}`];
     const newMetadata = processedData[`newmetadata_${field}`] 

src/middleware/uploadMiddleware.ts

@@ -158,8 +158,7 @@ async function processUploads(req: Request, res: Response, next: NextFunction) {
             }
 
             // Store in appropriate field
-            // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-            if (fieldName === 'newfile_Logo' || fieldName === 'newfile_BackgroundImage' || fieldName === 'newfile_MainImage' /* || fieldName === 'newfile_AccentGraphic' */) {
+            if (fieldName === 'newfile_Logo' || fieldName === 'newfile_BackgroundImage' || fieldName === 'newfile_MainImage') {
               uploadedAssets[fieldName] = asset;
             } else if (fieldName === 'newfile_PartnerLogos') {
               if (!uploadedAssets[fieldName]) {
@@ -208,11 +207,8 @@ const handleMultipartData = upload.fields([
   { name: 'newfile_Logo', maxCount: 1 },
   { name: 'newfile_BackgroundImage', maxCount: 1 },
   { name: 'newfile_MainImage', maxCount: 1 },
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // { name: 'newfile_AccentGraphic', maxCount: 1 },
   { name: 'newfile_PartnerLogos', maxCount: 5 },
   { name: 'newfile_ResourceFile', maxCount: 1 }
-  // SWEP banner image uses uploadSwepImage middleware
 ]);
 
 // Middleware for Banners

src/models/bannerModel.ts

@@ -1,12 +1,10 @@
-// TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
 import { 
   BannerTemplateType, 
   CharterType, 
   IBanner, 
   LayoutStyle, 
   TextColour, 
   UrgencyLevel,
-  // AccentGraphicSchema,
   BannerBackgroundSchema,
   CTAButtonSchema,
   DonationGoalSchema,
@@ -63,8 +61,6 @@ export const BannerSchema = new Schema({
   Logo: MediaAssetSchema,
   BackgroundImage: MediaAssetSchema,
   MainImage: MediaAssetSchema, // Separate image for split layout (not background)
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // AccentGraphic: AccentGraphicSchema,
 
   // Actions
   CtaButtons: {

src/schemas/bannerSchema.ts

@@ -1,8 +1,6 @@
 import { z } from 'zod';
 import {
   MediaAssetSchemaCore,
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // AccentGraphicSchemaCore,
   BannerBackgroundSchemaCore,
   CTAButtonSchemaCore,
   DonationGoalSchemaCore,
@@ -27,8 +25,6 @@ import { BannerTemplateType, UrgencyLevel, CharterType, LayoutStyle, TextColour,
 
 // API-specific schemas with preprocessing for FormData
 export const MediaAssetSchema = MediaAssetSchemaCore;
-// TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-// export const AccentGraphicSchema = AccentGraphicSchemaCore;
 export const BannerBackgroundSchema = BannerBackgroundSchemaCore;
 export const CTAButtonSchema = CTAButtonSchemaCore;
 

src/schemas/bannerSchemaCore.ts

@@ -113,8 +113,6 @@ export const BannerSchemaCore = z.object({
   Logo: MediaAssetSchemaCore,
   BackgroundImage: MediaAssetSchemaCore,
   MainImage: MediaAssetSchemaCore,
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // AccentGraphic: AccentGraphicSchemaCore,
 
   // Styling
   Background: BannerBackgroundSchemaCore,

src/types/banners/IAccentGraphic.ts

@@ -1,7 +1,5 @@
 import { Document, Types } from "mongoose";
 import { IMediaAsset } from "./IMediaAsset.js";
-// TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-// import { IAccentGraphic } from "./IAccentGraphic.js";
 import { IBannerBackground } from "./IBannerBackground.js";
 import { ICTAButton } from "./ICTAButton.js";
 import { IDonationGoal } from "./IDonationGoal.js";
@@ -41,8 +39,6 @@ export interface IBanner extends Document {
   Logo?: IMediaAsset;
   BackgroundImage?: IMediaAsset;
   MainImage?: IMediaAsset;
-  // TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-  // AccentGraphic?: IAccentGraphic;
 
   // Actions
   CtaButtons?: ICTAButton[];

src/types/banners/IBanner.ts

@@ -21,8 +21,6 @@ export * from './organisations/IServiceAddress.js';
 
 // Banner related types
 export * from './banners/IBanner.js';
-// TODO: Uncomment if AccentGraphic is needed. In the other case, remove.
-// export * from './IAccentGraphic.js';
 export * from './banners/IBannerBackground.js';
 export * from './banners/ICTAButton.js';
 export * from './banners/IDonationGoal.js';