refactor: update import fields (#433)

* refactor: update import fields

+ remove end message
+ add timer type
+ add end action

* chore: update tests

* version bump

apps/client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ontime-ui",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "private": true,
   "dependencies": {
     "@chakra-ui/react": "^2.5.5",

apps/electron/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ontime",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "author": "Carlos Valente",
   "description": "Time keeping for live events",
   "repository": "https://github.com/cpvalente/ontime",

apps/server/package.json

@@ -2,7 +2,7 @@
   "name": "ontime-server",
   "type": "module",
   "main": "src/index.ts",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "exports": "./src/index.js",
   "dependencies": {
     "body-parser": "^1.20.0",

apps/server/src/utils/__tests__/parser.test.js

@@ -587,10 +587,10 @@ describe('test parseExcel function', () => {
       ['Ontime ┬À Schedule Template'],
       [],
       ['Event Name', 'Test Event'],
-      ['Event URL', 'www.carlosvalente.com'],
+      ['Public URL', 'www.public.com'],
+      ['Backstage URL', 'www.backstage.com'],
       ['Public Info', 'test public info'],
       ['Backstage Info', 'test backstage info'],
-      ['End Message', 'test end message'],
       [],
       [],
       [
@@ -599,6 +599,8 @@ describe('test parseExcel function', () => {
         'Event Title',
         'Presenter Name',
         'Event Subtitle',
+        'End Action',
+        'Timer type',
         'Is Public? (x)',
         'Skip? (x)',
         'Notes',
@@ -620,6 +622,8 @@ describe('test parseExcel function', () => {
         'Guest Welcome',
         'Carlos',
         'Getting things started',
+        '',
+        '',
         'x',
         '',
         'Ballyhoo',
@@ -645,6 +649,8 @@ describe('test parseExcel function', () => {
         'A song from the hearth',
         'Still Carlos',
         'Derailing early',
+        'clock',
+        'load-next',
         '',
         '',
         'Rainbow chase',
@@ -669,11 +675,10 @@ describe('test parseExcel function', () => {
 
     const expectedParsedEvent = {
       title: 'Test Event',
-      publicUrl: '',
+      publicUrl: 'www.public.com',
-      backstageUrl: '',
+      backstageUrl: 'www.backstage.com',
       publicInfo: 'test public info',
       backstageInfo: 'test backstage info',
-      endMessage: 'test end message',
     };
 
     const expectedParsedRundown = [
@@ -683,6 +688,8 @@ describe('test parseExcel function', () => {
         title: 'Guest Welcome',
         presenter: 'Carlos',
         subtitle: 'Getting things started',
+        timerType: 'count-down',
+        endAction: 'none',
         isPublic: true,
         skip: false,
         note: 'Ballyhoo',
@@ -705,6 +712,8 @@ describe('test parseExcel function', () => {
         title: 'A song from the hearth',
         presenter: 'Still Carlos',
         subtitle: 'Derailing early',
+        timerType: 'clock',
+        endAction: 'load-next',
         isPublic: false,
         skip: true,
         note: 'Rainbow chase',

apps/server/src/utils/__tests__/parser.tests.js

@@ -1,878 +0,0 @@
-import { vi } from 'vitest';
-import { dbModel } from '../../models/dataModel.js';
-import { parseExcel, parseJson, validateEvent } from '../parser.ts';
-import { makeString, validateDuration } from '../parserUtils.js';
-import { parseAliases, parseUserFields, parseViewSettings } from '../parserFunctions.js';
-
-describe('test json parser with valid def', () => {
-  const testData = {
-    rundown: [
-      {
-        title: 'Guest Welcoming',
-        subtitle: '',
-        presenter: '',
-        note: '',
-        timeStart: 31500000,
-        timeEnd: 32400000,
-        timeType: 'start-end',
-        duration: 32400000 - 31500000,
-        isPublic: false,
-        colour: '',
-        type: 'event',
-        revision: 0,
-        id: '4b31',
-        user0: '',
-        user1: '',
-        user2: '',
-        user3: '',
-        user4: '',
-        user5: '',
-        user6: '',
-        user7: '',
-        user8: '',
-        user9: '',
-      },
-      {
-        title: 'Good Morning',
-        subtitle: 'Days schedule',
-        presenter: 'Carlos Valente',
-        note: '',
-        timeStart: 32400000,
-        timeEnd: 36000000,
-        timeType: 'start-end',
-        duration: 36000000 - 32400000,
-        isPublic: true,
-        skip: true,
-        colour: 'red',
-        type: 'event',
-        revision: 0,
-        id: 'f24d',
-        user0: '',
-        user1: '',
-        user2: '',
-        user3: '',
-        user4: '',
-        user5: '',
-        user6: '',
-        user7: '',
-        user8: '',
-        user9: '',
-      },
-      {
-        title: 'Stage 2 setup',
-        subtitle: '',
-        presenter: '',
-        note: '',
-        timeStart: 32400000,
-        timeEnd: 37200000,
-        timeType: 'start-end',
-        duration: 37200000 - 32400000,
-        isPublic: false,
-        colour: '',
-        type: 'event',
-        revision: 0,
-        id: 'bbc5',
-        user0: '',
-        user1: '',
-        user2: '',
-        user3: '',
-        user4: '',
-        user5: '',
-        user6: '',
-        user7: '',
-        user8: '',
-        user9: '',
-      },
-      {
-        title: 'Working Procedures',
-        subtitle: '',
-        presenter: 'Filip Johansen',
-        note: '',
-        timeStart: 37200000,
-        timeEnd: 39000000,
-        timeType: 'start-end',
-        duration: 39000000 - 37200000,
-        isPublic: true,
-        skip: false,
-        colour: '',
-        type: 'event',
-        revision: 0,
-        id: '5b3e',
-      },
-      {
-        title: 'Lunch',
-        subtitle: '',
-        presenter: '',
-        note: '',
-        timeStart: 39600000,
-        timeEnd: 45000000,
-        timeType: 'start-end',
-        duration: 37200000 - 32400000,
-        isPublic: false,
-        colour: '',
-        type: 'event',
-        revision: 0,
-        id: '8e2c',
-        user0: '',
-        user1: '',
-        user2: '',
-        user3: '',
-        user4: '',
-        user5: '',
-        user6: '',
-        user7: '',
-        user8: '',
-        user9: '',
-      },
-      {
-        title: 'A day being carlos',
-        subtitle: 'My life in a song',
-        presenter: 'Carlos Valente',
-        note: '',
-        timeStart: 46800000,
-        timeEnd: 50400000,
-        timeType: 'start-end',
-        duration: 37200000 - 32400000,
-        isPublic: true,
-        colour: '',
-        type: 'event',
-        revision: 0,
-        id: '08e9',
-        user0: '',
-        user1: '',
-        user2: '',
-        user3: '',
-        user4: '',
-        user5: '',
-        user6: '',
-        user7: '',
-        user8: '',
-        user9: '',
-      },
-      {
-        title: 'Hamburgers and Cheese',
-        subtitle: '... and other life questions',
-        presenter: 'Filip Johansen',
-        note: '',
-        timeStart: 54000000,
-        timeEnd: 57600000,
-        timeType: 'start-end',
-        duration: 37200000 - 32400000,
-        isPublic: true,
-        colour: '',
-        type: 'event',
-        revision: 0,
-        id: 'e25a',
-        user0: '',
-        user1: '',
-        user2: '',
-        user3: '',
-        user4: '',
-        user5: '',
-        user6: '',
-        user7: '',
-        user8: '',
-        user9: '',
-      },
-    ],
-    event: {
-      title: 'This is a test definition',
-      url: 'www.carlosvalente.com',
-      publicInfo: 'WiFi: demoproject \nPassword: ontimeproject',
-      backstageInfo: 'WiFi: demobackstage\nPassword: ontimeproject',
-    },
-    settings: {
-      app: 'ontime',
-      version: 2,
-      timeFormat: '24',
-    },
-  };
-
-  let parseResponse;
-
-  beforeEach(async () => {
-    parseResponse = await parseJson(testData);
-  });
-
-  it('has 7 events', () => {
-    const length = parseResponse?.rundown.length;
-    expect(length).toBe(7);
-  });
-
-  it('first event is as a match', () => {
-    const first = parseResponse?.rundown[0];
-    const expected = {
-      title: 'Guest Welcoming',
-      subtitle: '',
-      presenter: '',
-      note: '',
-      timeStart: 31500000,
-      timeEnd: 32400000,
-      timeType: 'start-end',
-      duration: 32400000 - 31500000,
-      isPublic: false,
-      skip: false,
-      colour: '',
-      type: 'event',
-      revision: 0,
-      id: '4b31',
-      user0: '',
-      user1: '',
-      user2: '',
-      user3: '',
-      user4: '',
-      user5: '',
-      user6: '',
-      user7: '',
-      user8: '',
-      user9: '',
-    };
-    expect(first).toStrictEqual(expected);
-  });
-
-  it('second event is as a match', () => {
-    const second = parseResponse?.rundown[1];
-    const expected = {
-      title: 'Good Morning',
-      subtitle: 'Days schedule',
-      presenter: 'Carlos Valente',
-      note: '',
-      timeStart: 32400000,
-      timeEnd: 36000000,
-      timeType: 'start-end',
-      duration: 36000000 - 32400000,
-      isPublic: true,
-      skip: true,
-      colour: 'red',
-      type: 'event',
-      revision: 0,
-      id: 'f24d',
-      user0: '',
-      user1: '',
-      user2: '',
-      user3: '',
-      user4: '',
-      user5: '',
-      user6: '',
-      user7: '',
-      user8: '',
-      user9: '',
-    };
-    expect(second).toStrictEqual(expected);
-  });
-
-  it('loaded event settings', () => {
-    const eventTitle = parseResponse?.event?.title;
-    expect(eventTitle).toBe('This is a test definition');
-  });
-
-  it('endMessage to exist but be empty', () => {
-    const endMessage = parseResponse?.event?.endMessage;
-    expect(endMessage).toBeDefined();
-    expect(endMessage).toBe('');
-  });
-
-  it('settings are for right app and version', () => {
-    const settings = parseResponse?.settings;
-    expect(settings.app).toBe('ontime');
-    expect(settings.version).toBe(2);
-  });
-
-  it('missing settings', () => {
-    const settings = parseResponse?.settings;
-    expect(settings.osc_port).toBeUndefined();
-  });
-});
-
-describe('test parser edge cases', () => {
-  it('generates missing ids', async () => {
-    const testData = {
-      rundown: [
-        {
-          title: 'Test Event',
-          type: 'event',
-        },
-      ],
-    };
-
-    const parseResponse = await parseJson(testData);
-    expect(parseResponse.rundown[0].id).toBeDefined();
-  });
-
-  it('detects duplicate Ids', async () => {
-    console.log = vi.fn();
-    const testData = {
-      rundown: [
-        {
-          title: 'Test Event 1',
-          type: 'event',
-          id: '1',
-        },
-        {
-          title: 'Test Event 2',
-          type: 'event',
-          id: '1',
-        },
-      ],
-    };
-
-    const parseResponse = await parseJson(testData);
-    expect(console.log).toHaveBeenCalledWith('ERROR: ID collision on import, skipping');
-    expect(parseResponse?.rundown.length).toBe(1);
-  });
-
-  it('handles incomplete datasets', async () => {
-    console.log = vi.fn();
-    const testData = {
-      rundown: [
-        {
-          title: 'Test Event 1',
-          id: '1',
-        },
-        {
-          title: 'Test Event 2',
-          id: '1',
-        },
-      ],
-    };
-
-    const parseResponse = await parseJson(testData);
-    expect(console.log).toHaveBeenCalledWith('ERROR: undefined event type, skipping');
-    expect(parseResponse?.rundown.length).toBe(0);
-  });
-
-  it('skips unknown app and version settings', async () => {
-    console.log = vi.fn();
-    const testData = {
-      settings: {
-        osc_port: 8888,
-      },
-    };
-
-    await parseJson(testData);
-    expect(console.log).toHaveBeenCalledWith('ERROR: unknown app version, skipping');
-  });
-});
-
-describe('test corrupt data', () => {
-  it('handles some empty events', async () => {
-    const emptyEvents = {
-      rundown: [
-        {},
-        {},
-        {},
-        {
-          title: 'Test Event 1',
-          type: 'event',
-          id: '1',
-        },
-        {
-          title: 'Test Event 2',
-          type: 'event',
-          id: '2',
-        },
-        {},
-        {},
-        {},
-      ],
-      event: {
-        title: 'All about Carlos demo event',
-        url: 'www.carlosvalente.com',
-        publicInfo: 'WiFi: demoproject \nPassword: ontimeproject',
-        backstageInfo: 'WiFi: demobackstage\nPassword: ontimeproject',
-        endMessage: '',
-      },
-      settings: {
-        app: 'ontime',
-        version: 2,
-        serverPort: 4001,
-        lock: null,
-        timeFormat: '24',
-      },
-    };
-
-    const parsedDef = await parseJson(emptyEvents);
-    expect(parsedDef.rundown.length).toBe(2);
-  });
-
-  it('handles all empty events', async () => {
-    const emptyEvents = {
-      rundown: [{}, {}, {}, {}, {}, {}, {}, {}],
-      event: {
-        title: 'All about Carlos demo event',
-        url: 'www.carlosvalente.com',
-        publicInfo: 'WiFi: demoproject \nPassword: ontimeproject',
-        backstageInfo: 'WiFi: demobackstage\nPassword: ontimeproject',
-        endMessage: '',
-      },
-      settings: {
-        app: 'ontime',
-        version: 2,
-        serverPort: 4001,
-        lock: null,
-        timeFormat: '24',
-      },
-    };
-
-    const parsedDef = await parseJson(emptyEvents);
-    expect(parsedDef.rundown.length).toBe(0);
-  });
-
-  it('handles missing event data', async () => {
-    const emptyEventData = {
-      rundown: [{}, {}, {}, {}, {}, {}, {}, {}],
-      event: {},
-      settings: {
-        app: 'ontime',
-        version: 2,
-        serverPort: 4001,
-        lock: null,
-        timeFormat: '24',
-      },
-    };
-
-    const parsedDef = await parseJson(emptyEventData);
-    expect(parsedDef.event).toStrictEqual(dbModel.event);
-  });
-
-  it('handles missing settings', async () => {
-    const missingSettings = {
-      rundown: [{}, {}, {}, {}, {}, {}, {}, {}],
-      event: {},
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-    };
-
-    const parsedDef = await parseJson(missingSettings);
-    expect(parsedDef.settings).toStrictEqual(dbModel.settings);
-  });
-
-  it('fails with invalid JSON', async () => {
-    console.log = vi.fn();
-    const invalidJSON = 'some random dataset';
-    const parsedDef = await parseJson(invalidJSON);
-    expect(console.log).toHaveBeenCalledWith('ERROR: Invalid JSON format');
-    expect(parsedDef).toBe(-1);
-  });
-});
-
-describe('test event validator', () => {
-  it('validates a good object', () => {
-    const event = {
-      title: 'test',
-    };
-    const validated = validateEvent(event);
-
-    expect(validated).toEqual(
-      expect.objectContaining({
-        title: expect.any(String),
-        subtitle: expect.any(String),
-        presenter: expect.any(String),
-        note: expect.any(String),
-        timeStart: expect.any(Number),
-        timeEnd: expect.any(Number),
-        isPublic: expect.any(Boolean),
-        skip: expect.any(Boolean),
-        revision: expect.any(Number),
-        type: expect.any(String),
-        id: expect.any(String),
-        colour: expect.any(String),
-        user0: expect.any(String),
-        user1: expect.any(String),
-        user2: expect.any(String),
-        user3: expect.any(String),
-        user4: expect.any(String),
-        user5: expect.any(String),
-        user6: expect.any(String),
-        user7: expect.any(String),
-        user8: expect.any(String),
-        user9: expect.any(String),
-      }),
-    );
-  });
-
-  it('fails an empty object', () => {
-    const event = {};
-    const validated = validateEvent(event);
-    expect(validated).toEqual(null);
-  });
-
-  it('makes objects strings', () => {
-    const event = {
-      title: 2,
-      subtitle: true,
-      presenter: 3.2,
-      note: '1899-12-30T08:00:10.000Z',
-    };
-    const validated = validateEvent(event);
-    expect(typeof validated.title).toEqual('string');
-    expect(typeof validated.subtitle).toEqual('string');
-    expect(typeof validated.presenter).toEqual('string');
-    expect(typeof validated.note).toEqual('string');
-  });
-
-  it('enforces numbers on times', () => {
-    const event = {
-      timeStart: false,
-      timeEnd: '2',
-    };
-    const validated = validateEvent(event);
-    expect(typeof validated.timeStart).toEqual('number');
-    expect(validated.timeStart).toEqual(0);
-    expect(typeof validated.timeEnd).toEqual('number');
-    expect(validated.timeEnd).toEqual(0);
-  });
-
-  it('handles bad objects', () => {
-    const event = {
-      title: {},
-    };
-    const validated = validateEvent(event);
-    expect(typeof validated.title).toEqual('string');
-  });
-});
-
-describe('test makeString function', () => {
-  it('converts variables to string', () => {
-    let val = 2;
-    let expected = '2';
-    let converted = makeString(val);
-    expect(converted).toBe(expected);
-
-    val = 2.22222222;
-    expected = '2.22222222';
-    converted = makeString(val);
-    expect(converted).toBe(expected);
-
-    val = ['testing'];
-    expected = 'testing';
-    converted = makeString(val);
-    expect(converted).toBe(expected);
-
-    val = { doing: 'testing' };
-    converted = makeString(val, 'fallback');
-    expect(converted).toBe('fallback');
-  });
-});
-
-describe('test parseExcel function', () => {
-  it('parses the example file', async () => {
-    const testdata = [
-      ['Ontime ┬À Schedule Template'],
-      [],
-      ['Event Name', 'Test Event'],
-      ['Event URL', 'www.carlosvalente.com'],
-      ['Public Info', 'test public info'],
-      ['Backstage Info', 'test backstage info'],
-      ['End Message', 'test end message'],
-      [],
-      [],
-      [
-        'Time Start',
-        'Time End',
-        'Event Title',
-        'Presenter Name',
-        'Event Subtitle',
-        'Is Public? (x)',
-        'Skip? (x)',
-        'Notes',
-        'User0:test0',
-        'User1:test1',
-        'User2:test2',
-        'User3:test3',
-        'User4:test4',
-        'User5:test5',
-        'User6:test6',
-        'user7:test7',
-        'user8:test8',
-        'user9:test9',
-        'Colour',
-      ],
-      [
-        '1899-12-30T07:00:00.000Z',
-        '1899-12-30T08:00:10.000Z',
-        'Guest Welcome',
-        'Carlos',
-        'Getting things started',
-        'x',
-        '',
-        'Ballyhoo',
-        '',
-        '',
-        '',
-        '',
-        'a0',
-        'a1',
-        'a2',
-        'a3',
-        'a4',
-        'a5',
-        'a6',
-        'a7',
-        'a8',
-        'a9',
-        'red',
-      ],
-      [
-        '1899-12-30T08:00:00.000Z',
-        '1899-12-30T08:30:00.000Z',
-        'A song from the hearth',
-        'Still Carlos',
-        'Derailing early',
-        '',
-        '',
-        'Rainbow chase',
-        '',
-        '',
-        '',
-        '',
-        'b0',
-        '',
-        '',
-        '',
-        '',
-        'b5',
-        '',
-        '',
-        '',
-        '',
-        '#F00',
-      ],
-      [],
-    ];
-
-    const expectedParsedEvent = {
-      title: 'Test Event',
-      url: 'www.carlosvalente.com',
-      publicInfo: 'test public info',
-      backstageInfo: 'test backstage info',
-      endMessage: 'test end message',
-    };
-
-    const expectedParsedRundown = [
-      {
-        timeStart: 25200000,
-        timeEnd: 28810000,
-        title: 'Guest Welcome',
-        presenter: 'Carlos',
-        subtitle: 'Getting things started',
-        isPublic: true,
-        skip: false,
-        note: 'Ballyhoo',
-        user0: 'a0',
-        user1: 'a1',
-        user2: 'a2',
-        user3: 'a3',
-        user4: 'a4',
-        user5: 'a5',
-        user6: 'a6',
-        user7: 'a7',
-        user8: 'a8',
-        user9: 'a9',
-        colour: 'red',
-        type: 'event',
-      },
-      {
-        timeStart: 28800000,
-        timeEnd: 30600000,
-        title: 'A song from the hearth',
-        presenter: 'Still Carlos',
-        subtitle: 'Derailing early',
-        isPublic: false,
-        skip: true,
-        note: 'Rainbow chase',
-        user0: 'b0',
-        user5: 'b5',
-        colour: '#F00',
-        type: 'event',
-      },
-    ];
-
-    const parsedData = await parseExcel(testdata);
-
-    expect(parsedData.event).toStrictEqual(expectedParsedEvent);
-    expect(parsedData.rundown).toBeDefined();
-    expect(parsedData.rundown.title).toBe(expectedParsedRundown.title);
-    expect(parsedData.rundown.presenter).toBe(expectedParsedRundown.presenter);
-    expect(parsedData.rundown.subtitle).toBe(expectedParsedRundown.subtitle);
-    expect(parsedData.rundown.isPublic).toBe(expectedParsedRundown.isPublic);
-    expect(parsedData.rundown.skip).toBe(expectedParsedRundown.skip);
-    expect(parsedData.rundown.note).toBe(expectedParsedRundown.note);
-    expect(parsedData.rundown.type).toBe(expectedParsedRundown.type);
-  });
-});
-
-describe('test aliases import', () => {
-  it('imports a well defined alias', () => {
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-      aliases: [
-        {
-          enabled: false,
-          alias: 'testalias',
-          pathAndParams: 'testpathAndParams',
-        },
-      ],
-    };
-
-    const parsed = parseAliases(testData);
-    expect(parsed.length).toBe(1);
-
-    // generates missing id
-    expect(parsed[0].id).toBeDefined();
-  });
-});
-
-describe('test userFields import', () => {
-  const model = dbModel.userFields;
-  it('imports a fully defined user fields', () => {
-    const testUserFields = {
-      user0: 'test0',
-      user1: 'test1',
-      user2: 'test2',
-      user3: 'test3',
-      user4: 'test4',
-      user5: 'test5',
-      user6: 'test6',
-      user7: 'test7',
-      user8: 'test8',
-      user9: 'test9',
-    };
-
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-      userFields: testUserFields,
-    };
-
-    const parsed = parseUserFields(testData);
-    expect(parsed).toStrictEqual(testUserFields);
-  });
-
-  it('imports a partially defined user fields', () => {
-    const testUserFields = {
-      user0: 'test0',
-      user1: 'test1',
-      user7: 'test7',
-      user8: 'test8',
-      user9: 'test9',
-    };
-
-    const expected = {
-      ...model,
-      ...testUserFields,
-    };
-
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-      userFields: testUserFields,
-    };
-
-    const parsed = parseUserFields(testData);
-    expect(parsed).toStrictEqual(expected);
-  });
-
-  it('handles missing user fields', () => {
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-    };
-
-    const parsed = parseUserFields(testData);
-    expect(parsed).toStrictEqual(model);
-    expect(parsed).toStrictEqual(model);
-  });
-
-  it('ignores badly defined fields', () => {
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-      userFields: {
-        notThis: 'this shouldng be accepted',
-        orThis: 'this neither',
-      },
-    };
-
-    const parsed = parseUserFields(testData);
-    expect(parsed).toStrictEqual(model);
-  });
-});
-
-describe('test views import', () => {
-  it('imports data from file', () => {
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-      views: {
-        overrideStyles: true,
-      },
-    };
-    const parsed = parseViewSettings(testData);
-    expect(parsed).toStrictEqual(testData.views);
-  });
-
-  it('imports defaults to model', () => {
-    const testData = {
-      rundown: [],
-      settings: {
-        app: 'ontime',
-        version: 2,
-      },
-    };
-    const parsed = parseViewSettings(testData, true);
-    expect(parsed).toStrictEqual(dbModel.views);
-  });
-});
-
-describe('test validateDuration()', () => {
-  describe('handles valid inputs', () => {
-    const valid = [
-      { test: 'zero values', timeStart: 0, timeEnd: 0 },
-      { test: 'end after start', timeStart: 0, timeEnd: 1 },
-    ];
-
-    valid.forEach((t) => {
-      it(t.test, () => {
-        const d = validateDuration(t.timeStart, t.timeEnd);
-        expect(d).toBe(t.timeEnd - t.timeStart);
-      });
-    });
-  });
-
-  describe('handles edge cases', () => {
-    // edge cases
-    const testData = [
-      { test: 'negative 0', timeStart: -0, timeEnd: -0, expected: 0 },
-      { test: 'end before start', timeStart: 2, timeEnd: 1, expected: 0 },
-    ];
-
-    testData.forEach((t) => {
-      it(t.test, () => {
-        const d = validateDuration(t.timeStart, t.timeEnd);
-        expect(d).toBe(t.expected);
-      });
-    });
-  });
-});

apps/server/src/utils/parser.ts

@@ -65,7 +65,6 @@ export const parseExcel = async (excelData) => {
       let publicInfoNext = false;
       let backstageUrlNext = false;
       let backstageInfoNext = false;
-      let endMessageNext = false;
 
       const event: Partial<OntimeEvent> = {};
 
@@ -81,14 +80,11 @@ export const parseExcel = async (excelData) => {
           eventData.publicInfo = column;
           publicInfoNext = false;
         } else if (backstageUrlNext) {
-          eventData.publicUrl = column;
+          eventData.backstageUrl = column;
           backstageUrlNext = false;
         } else if (backstageInfoNext) {
           eventData.backstageInfo = column;
           backstageInfoNext = false;
-        } else if (endMessageNext) {
-          eventData.endMessage = column;
-          endMessageNext = false;
         } else if (j === timeStartIndex) {
           event.timeStart = parseExcelDate(column);
         } else if (j === timeEndIndex) {
@@ -152,9 +148,6 @@ export const parseExcel = async (excelData) => {
               case 'backstage info':
                 backstageInfoNext = true;
                 break;
-              case 'end message':
-                endMessageNext = true;
-                break;
               case 'time start':
               case 'start':
                 timeStartIndex = j;
@@ -187,6 +180,7 @@ export const parseExcel = async (excelData) => {
               case 'skip':
                 skipIndex = j;
                 break;
+              case 'note':
               case 'notes':
                 notesIndex = j;
                 break;
@@ -254,7 +248,7 @@ export const parseExcel = async (excelData) => {
     });
   return {
     rundown,
-    eventData: eventData,
+    eventData,
     settings: {
       app: 'ontime',
       version: 2,

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ontime",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "description": "Time keeping for live events",
   "keywords": [
     "lighdev",

packages/utils/package.json

@@ -2,7 +2,7 @@
   "name": "ontime-utils",
   "type": "module",
   "exports": "./index.ts",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "private": true,
   "description": "shared logic for ontime",
   "scripts": {