Remove the concept of talk slugs

config/default.yaml

@@ -42,10 +42,10 @@ livestream:
 
   # The template for livestreams in talk rooms
   # Available variables:
-  #  slug  - The shorthand version of the talk name (eg: "matrix-p2p")
+  #  id    - The talk's ID
   #  audId - The auditorium ID the talk is held in (eg: "D.collab")
   #  jitsi - The Jitsi conference name in lowercase.
-  talkUrl: "https://stream.example.org/conference-talks/hls/{slug}.m3u8"
+  talkUrl: "https://stream.example.org/conference-talks/hls/talk-{id}.m3u8"
 
   # The template for livestreams in random rooms.
   # Available variables:
@@ -255,4 +255,4 @@ metrics:
   address: 127.0.0.1
 
 # if set to `true` will prevent the conf-bot from sending live invites to email/matrix_ids
-dry_run_enabled: false
+dry_run_enabled: false

src/Conference.ts

@@ -635,7 +635,8 @@ export class Conference {
 
         // Calculate all the aliases the room should have, then update the list of bot-assigned aliases to match
         const wantedBaseNames = [
-            (await auditorium.getSlug()) + '-' + talk.slug,
+            // Talk slugs no longer exist. But if we ever use this feature again, we probably want to reinstate them.
+            // (await auditorium.getSlug()) + '-' + talk.slug,
             'talk-' + talk.id,
         ];
         const wantedPrefixedNames = wantedBaseNames.flatMap(baseName => applyAllAliasPrefixes(baseName, this.config.conference.prefixes.aliases));

src/__tests__/backends/penta/__snapshots__/PentabarfParser.test.ts.snap

@@ -18,7 +18,6 @@ exports[`parsing pentabarf XML: overview: auditoriums 1`] = `
         "prerecorded": true,
         "pretalxCode": undefined,
         "qa_startTime": null,
-        "slug": "testcon_opening_remarks",
         "speakers": [
           {
             "email": "",
@@ -64,7 +63,6 @@ exports[`parsing pentabarf XML: overview: conference 1`] = `
           "prerecorded": true,
           "pretalxCode": undefined,
           "qa_startTime": null,
-          "slug": "testcon_opening_remarks",
           "speakers": [
             {
               "email": "",
@@ -126,7 +124,6 @@ exports[`parsing pentabarf XML: overview: talks 1`] = `
     "prerecorded": true,
     "pretalxCode": undefined,
     "qa_startTime": null,
-    "slug": "testcon_opening_remarks",
     "speakers": [
       {
         "email": "",
@@ -162,7 +159,6 @@ exports[`tracks that set a online qa value correctly apply to talks: talks 1`] =
     "prerecorded": true,
     "pretalxCode": undefined,
     "qa_startTime": 0,
-    "slug": "testcon_opening_remarks",
     "speakers": [
       {
         "email": "",
@@ -193,7 +189,6 @@ exports[`tracks that set a online qa value correctly apply to talks: talks 1`] =
     "prerecorded": true,
     "pretalxCode": undefined,
     "qa_startTime": 0,
-    "slug": "testcon_opening_remarks",
     "speakers": [
       {
         "email": "",
@@ -224,7 +219,6 @@ exports[`tracks that set a online qa value correctly apply to talks: talks 1`] =
     "prerecorded": true,
     "pretalxCode": undefined,
     "qa_startTime": null,
-    "slug": "testcon_opening_remarks",
     "speakers": [
       {
         "email": "",
@@ -255,7 +249,6 @@ exports[`tracks that set a online qa value correctly apply to talks: talks 1`] =
     "prerecorded": true,
     "pretalxCode": undefined,
     "qa_startTime": 0,
-    "slug": "testcon_opening_remarks",
     "speakers": [
       {
         "email": "",

src/backends/json/JsonScheduleLoader.ts

@@ -68,7 +68,6 @@ export class JsonScheduleLoader {
             id: talk.id.toString(), // TODO We have numbers on the rhs and a string on the lhs.
             title: talk.title,
             subtitle: talk.description, // TODO is this valid?
-            slug: slugify(talk.title),
 
             auditoriumId,
             prerecorded: true, // TODO
@@ -108,4 +107,4 @@ export class JsonScheduleLoader {
             isPhysical: false,
         };
     }
-}
+}

src/backends/penta/PentabarfParser.ts

@@ -246,7 +246,6 @@ export class PentabarfParser {
                         dateTs: dateTs,
                         startTime: startTime.valueOf(),
                         endTime: endTime.valueOf(),
-                        slug: pEvent.slug,
                         title: pEvent.title,
                         subtitle: pEvent.subtitle,
                         track: pEvent.track,

src/backends/pretalx/PretalxParser.ts

@@ -195,7 +195,6 @@ export async function parseFromJSON(rawJson: string, prefixConfig: IPrefixConfig
                         })), //event.persons,
                         // TODO: Unsure?
                         auditoriumId: roomName,
-                        slug: event.slug,
                     };
                     talks.set(eventCode, talk);
                     auditorium?.talks.set(eventCode, talk);
@@ -210,4 +209,4 @@ export async function parseFromJSON(rawJson: string, prefixConfig: IPrefixConfig
         auditoriums,
         talks,
     }
-}
+}

src/models/schedule.ts

@@ -52,7 +52,6 @@ export interface ITalk {
      */
     qa_startTime: number | null;
     livestream_endTime: number;
-    slug: string;
     title: string;
     subtitle: string;
     track: string;

src/web.ts

@@ -81,7 +81,7 @@ export async function renderTalkWidget(req: Request, res: Response, conference:
 
     const streamUrl = template(talkUrl, {
         audId: audId.toLowerCase(),
-        slug: (await talk.getDefinition()).slug.toLowerCase(),
+        id: await talk.getId(),
         jitsi: base32.stringify(Buffer.from(talk.roomId), { pad: false }).toLowerCase(),
     });