Automatic and Manual Event Ending by mikewillems · Pull Request #417 · berkmancenter/frankly

mikewillems · 2026-05-02T00:24:52Z

Automatic and Manual Event Ending

What is in this PR?

Establish and test a consistent mechanism for ending an event manually and at a particular time. Enable the host to close all meeting rooms manually with a button or automatically at end time (with an optional grace period before closing all rooms). Add and test the UI for this functionality, and limit the grace period to an integer number of minutes between 0 (default) and 120.

Changes in the codebase

General Approach

The core concept is a meetingEndedAt timestamp on the LiveMeeting document. Setting this field triggers two things: (1) the client detects the change via its live meeting stream and auto-leaves, and (2) server-side cleanup runs (stop recordings, send "event ended" email).

Two server paths can set meetingEndedAt:

Manual: The host clicks "End Meeting" in the admin panel, which calls the endMeetingForAll Cloud Function. Authorization checks confirm the caller is the event creator or a community mod/admin/owner.
Automatic: On first join, GetMeetingJoinInfo schedules a Cloud Task for scheduledTime + durationInMinutes + gracePeriod. When the task fires, ScheduledEndMeeting sets meetingEndedAt via the same shared core logic.

Both paths use a Firestore transaction for the idempotency guard: if meetingEndedAt is already set, the operation is a no-op. This prevents duplicate emails and recording stops even if the manual and scheduled paths race.

On the client, the event settings drawer exposes the autoEndMeeting toggle and a grace period input field. The grace period is validated client-side (0-120 integer, with debounced auto-correction) and clamped server-side as a defense-in-depth measure.

Non-obvious choices

Grace period input uses a managed TextEditingController rather than TextFormField with initialValue. The naive approach (using a ValueKey keyed to the model value) causes Flutter to destroy and recreate the TextFormField widget on every keystroke, because onChanged -> updateSettingValue -> setState changes the key. This also unfocuses the field. The managed controller approach avoids this by keeping the controller alive across rebuilds, with a _gracePeriodEditing flag to prevent updateView from overwriting the text while the user is typing.
Auto-end task scheduling is best-effort (try/catch). A Cloud Tasks failure during GetMeetingJoinInfo would otherwise block the user from joining the meeting. The task is non-critical -- the host can always end manually.
ScheduledEndMeeting validates the X-CloudTasks-TaskName header. Cloud Functions strips this header from external HTTP requests, so its presence confirms the call originated from a Cloud Tasks queue. This prevents unauthorized callers from ending meetings via the raw HTTP endpoint.
Recording stops and email delivery run in parallel. These are independent operations. Running them sequentially risked approaching the Cloud Function timeout for events with many breakout rooms and gives inconsistent user experience (not that it really matters in this case but polish is nice).
Dialog dismiss is treated as cancel in the host leave prompt. On my first pass, dismissing the "End for all?" dialog (tapping outside) forced the result to false and silently left the meeting. Now it cancels the leave action entirely, requiring an explicit choice.

Changes by Commit

7a3b4310 -- Add meetingEndedAt field to LiveMeeting, create endMeetingForAll CF with auth checks, extract recording stop logic into stop_all_event_recordings.dart
34cc52d1 -- Add unit tests for EndMeetingForAll CF
b190f08a -- Client reacts to meetingEndedAt changes (auto-leave), wire up endMeetingForAll call, add "End Meeting" button in admin panel
547212af -- Add ScheduledEndMeeting CF (Cloud Tasks HTTP handler), schedule auto-end on first join in GetMeetingJoinInfo
c4c34713 -- Add unit tests for ScheduledEndMeeting CF
29107097 -- Comment _postEventEmailThresholdInMinutes explaining its purpose
887a9e1c -- Display end time alongside start time in event UI (carousel, event cards, event info)
4fc78c4d -- Add widget tests for VerticalTimeAndDateIndicator end time display
5992828e -- Reject join attempts on ended meetings (server-side check in GetMeetingJoinInfo)
aa1fde93 -- Add test for join rejection on ended meetings
b2ef50a8 -- Add host leave prompt ("End for all?" dialog before leaving)
62bf5e14 -- Add autoEndMeeting and autoEndGracePeriodMinutes fields to EventSettings, add toggle + grace period input to event settings drawer, wire into task scheduling
b42c2bd5 -- Fix settings type error by filtering non-bool values from the dev settings bool display
4e27021f -- Wrap the meetingEndedAt read-then-write in a Firestore transaction for atomic idempotency (both EndMeetingForAll and ScheduledEndMeeting)
9a3b0c7b -- Remove dead displayName resolution code and unused imports from GetMeetingJoinInfo
2a3df808 -- Pass event from the transaction result via MeetingJoinResult instead of re-reading it from Firestore
97ee210c -- Wrap auto-end task scheduling in try/catch so join is not blocked by Cloud Tasks failures
09e4eb52 -- Localize hard-coded strings in leave prompt and event settings (ARB files)
0ed29a9b -- Validate X-CloudTasks-TaskName header in ScheduledEndMeeting to prevent unauthorized external calls
b42c8d3f -- Add email verification to scheduled end meeting test
295e7db5 -- Clamp grace period to 0-120 minutes server-side in GetMeetingJoinInfo
b56e9a01 -- Add grace period input validation and auto-correction (managed controller, debounce, red error styling)
e428a3fe -- Extract shared end meeting core logic (transaction, recording stop, email) into end_meeting_core.dart, called by both CFs
5f89aa07 -- Treat dialog dismiss as cancel in host leave prompt
aafef468 -- Parallelize breakout room recording stops with Future.wait
72f7f3e4 -- Run recording stops in parallel with email delivery in end meeting core
0938e944 -- Use permissionDenied instead of failedPrecondition for auth errors (better error categorization in logs)
a978fdcb -- Short-circuit auth check for event creator in endMeetingForAll (skip membership lookup when the caller is the host)
d3b0dcd9 -- Guard against double-tap on leave with _leavingInProgress flag
3b6f104d -- Add widget tests for grace period input validation and auto-correction
cb49d913 -- Add comments explaining the validation scheme

Changes outside the codebase

No infrastructure or environment changes. Cloud Tasks is already enabled in the project. The ScheduledEndMeeting function is registered in main.dart and deployed alongside other Cloud Functions.

Testing this PR

Automated tests:

EndMeetingForAll CF unit tests (end_meeting_for_all_test.dart) -- covers auth, idempotency, recording stop, email send
ScheduledEndMeeting CF unit tests (scheduled_end_meeting_test.dart) -- covers execution, idempotency, email verify
GetMeetingJoinInfo test for join rejection on ended meetings
Widget tests for end time display in VerticalTimeAndDateIndicator
Widget tests for grace period input (initial value, focus retention, error styling, debounce auto-correction, focus loss correction, error clearing)

Run CF tests:

cd firebase/functions && CLOUD_RUNTIME_CONFIG=./test/test_config.json \
  firebase emulators:exec --only firestore --project frankly-dev-a410b \
  'dart test --concurrency=1'

Run client tests:

cd client && .fvm/flutter_sdk/bin/flutter test --platform chrome

Manual testing checklist:

Toggle "Auto-end meeting" on in event settings, set a grace period, save, and verify the meeting auto-ends at the scheduled time
Click "End Meeting" in the admin panel during a live meeting and verify all participants are ejected
Verify the "End for all?" prompt appears when the host clicks Leave, and that dismissing it cancels the leave action
Type invalid values (negative, >120, non-numeric, decimal) in the grace period field and verify red styling + auto-correction
Join an ended meeting and verify the join is rejected

Additional information

The meetingEndedAt field is write-once. Once set, there is no mechanism to "restart" a meeting. This is intentional -- ending is a final action confirmed by a dialog.

The grace period field is clamped to 0-120 on both client and server. The server-side clamp in GetMeetingJoinInfo is an extra defense (and good hygeine in case of unvalidated input attacks + is coupled with authentication); the client-side validation is the primary UX.

The ScheduledEndMeeting Cloud Function is an HTTP handler (not a callable) because Cloud Tasks dispatches via HTTP POST. The X-CloudTasks-TaskName header check is the auth mechanism -- Cloud Functions infrastructure strips this header from external requests.

Only conflict in client/lib/features/admin/presentation/views/data_tab.dart

- Verify meetingEndedAt is written and email sent - Verify idempotency (second call is a no-op) - Verify non-mod/admin/owner is rejected

- Auto-leave when meetingEndedAt is set on LiveMeeting doc - Add endMeetingForAll to CloudFunctionsEventService - Add End Meeting menu item in admin panel for mods/admins/owners - Show confirmation dialog before ending - Add l10n strings in all 4 ARB files (en, es, zh, zh_Hant_TW)

- Add ScheduledEndMeeting OnRequestMethod CF with idempotency guard - Add isFirstJoin flag to MeetingJoinResult - Schedule end task on first join when event has scheduledTime + duration - Register ScheduledEndMeeting in main.dart

- Verify meetingEndedAt is written and email sent - Verify idempotency (second call is a no-op)

# Conflicts: # client/lib/features/admin/presentation/views/data_tab.dart

- Add optional endTime parameter to VerticalTimeAndDateIndicator - Show time range (e.g. '2:30p - 3:30p') when endTime is provided - Pass calculated endTime (scheduledTime + durationInMinutes) at all 5 call sites: event_card, carousel_tabs, home_page_event_card, event_button, event_info

- Verify start-only display - Verify start-end range display

data_models/build.sh + firebase/functions/build.sh -> build-all.sh Expanded script to cover all three packages (client, data_models, firebase/functions) and use the FVM-pinned Flutter/Dart SDK instead of global binaries, fixing kernel binary format mismatch errors when the global SDK version differs from the repo pin.

Resolves kernel binary format mismatch errors caused by global Flutter/Dart differing from the repo-pinned version. All flutter/dart invocations now prefer client/.fvm/flutter_sdk/bin, with fallback to global. Also clears build stamps when the active SDK fingerprint changes between runs.

- Add meetingEndedAt (DateTime?) and kFieldMeetingEndedAt to LiveMeeting - Add EndMeetingForAllRequest to data_models requests - Break out shared recording-stop logic into stop_all_event_recordings.dart - Refactor EventEnded CF to use shared helper - Add EndMeetingForAll callable CF with mod/admin/owner auth check, idempotency guard, recording stop, and post-event email - Register EndMeetingForAll in main.dart

github-actions · 2026-05-02T00:31:57Z

Visit the preview URL for this PR (updated for commit f696265):

https://gen-hls-bkc-7627--pr417-mw-feat-event-end-y4w13zyo.web.app

_{(expires Thu, 21 May 2026 00:07:37 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: eed668cca81618d491d024574a8f8a6003deaa8d}

Copilot

Pull request overview

This PR introduces a host-initiated and scheduled mechanism to end live meetings, including persisting an “ended” marker on the LiveMeeting document, stopping active recordings, and notifying participants. It also updates client UI to display event end times and exposes “End Meeting” controls, plus adds dev/build script improvements.

Changes:

Adds Cloud Functions to end a meeting for all participants (manual, host/mod/admin/owner) and to end it automatically at a scheduled time (Cloud Tasks).
Persists meetingEndedAt in LiveMeeting, blocks new joins after end, and auto-leaves clients when the meeting is ended.
Updates UI to show time ranges (start–end), adds event settings for auto-end + grace period, and adds build/dev scripting improvements.

Reviewed changes

Copilot reviewed 40 out of 40 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
run-dev.sh	Uses FVM Flutter/Dart when available; clears build stamps when SDK fingerprint changes.
package.json	Adds `build` script entrypoint.
build-all.sh	New build script to run pub get/build steps across client/data_models/functions.
firebase/functions/node/main.dart	Registers new EndMeetingForAll (onCall) and ScheduledEndMeeting (onRequest) functions.
firebase/functions/lib/events/live_meetings/end_meeting_for_all.dart	Implements callable “end meeting for all” flow (writes `meetingEndedAt`, stops recordings, emails participants).
firebase/functions/lib/events/live_meetings/scheduled_end_meeting.dart	Implements scheduled HTTP function to end meeting at a given time.
firebase/functions/lib/events/live_meetings/stop_all_event_recordings.dart	Extracts shared logic to stop main + breakout recordings.
firebase/functions/lib/events/live_meetings/get_meeting_join_info.dart	Rejects joining ended meetings; schedules auto-end task on first join.
firebase/functions/lib/events/live_meetings/live_meeting_utils.dart	Returns `isFirstJoin` flag to support first-join scheduling.
firebase/functions/lib/events/event_ended.dart	Reuses shared “stop all recordings” helper.
firebase/functions/test/events/live_meetings/end_meeting_for_all_test.dart	Adds tests for manual end meeting behavior, idempotency, and authorization.
firebase/functions/test/events/live_meetings/scheduled_end_meeting_test.dart	Adds tests for scheduled meeting end behavior and idempotency.
firebase/functions/test/events/live_meetings/get_meeting_join_info_test.dart	Adds test that join is rejected if `meetingEndedAt` is set.
data_models/lib/events/live_meetings/live_meeting.dart	Adds `meetingEndedAt` field + Firestore key constant.
data_models/lib/events/live_meetings/live_meeting.g.dart	Generated JSON serialization updates for `meetingEndedAt`.
data_models/lib/events/live_meetings/live_meeting.freezed.dart	Generated Freezed updates for `meetingEndedAt`.
data_models/lib/events/event.dart	Adds `autoEndMeeting` + `autoEndGracePeriodMinutes` settings keys/defaults.
data_models/lib/events/event.g.dart	Generated JSON serialization updates for new EventSettings fields.
data_models/lib/events/event.freezed.dart	Generated Freezed updates for new EventSettings fields.
data_models/lib/cloud_functions/requests.dart	Adds `EndMeetingForAllRequest`.
data_models/lib/cloud_functions/requests.g.dart	Generated JSON serialization for `EndMeetingForAllRequest`.
data_models/lib/cloud_functions/requests.freezed.dart	Generated Freezed code for `EndMeetingForAllRequest`.
client/lib/features/events/data/services/cloud_functions_event_service.dart	Adds client call for `endMeetingForAll`.
client/lib/features/events/features/live_meeting/features/admin_panel/presentation/widgets/admin_panel.dart	Adds “End Meeting” menu item + confirmation dialog.
client/lib/features/events/features/live_meetings/data/providers/live_meeting_provider.dart	Auto-leaves when meeting ended; prompts host/admin to optionally end for all when leaving.
client/lib/features/events/features/event_page/presentation/views/event_settings_drawer.dart	Adds UI for auto-end meeting toggle + grace period input.
client/lib/features/events/features/event_page/presentation/event_settings_presenter.dart	Adds `updateSettingValue` to support non-bool settings updates.
client/lib/features/community/presentation/widgets/carousel/time_indicator.dart	Displays time range when `endTime` is provided.
client/test/lib/features/community/presentation/widgets/carousel/time_indicator_test.dart	Adds widget tests for time-range display behavior.
client/lib/features/community/presentation/widgets/carousel/carousel_tabs.dart	Supplies `endTime` to time indicator in carousel.
client/lib/features/community/presentation/widgets/event_card.dart	Supplies `endTime` to time indicator.
client/lib/features/home/presentation/views/home_page_event_card.dart	Supplies `endTime` to time indicator.
client/lib/features/events/presentation/widgets/event_button.dart	Supplies `endTime` to time indicator.
client/lib/features/events/features/event_page/presentation/widgets/event_info.dart	Supplies `endTime` to time indicator.
client/lib/l10n/app_en.arb	Adds `endMeeting` and `endMeetingConfirmation` strings.
client/lib/l10n/app_es.arb	Adds `endMeeting` and `endMeetingConfirmation` strings.
client/lib/l10n/app_zh.arb	Adds `endMeeting` and `endMeetingConfirmation` strings.
client/lib/l10n/app_zh_Hant_TW.arb	Adds `endMeeting` and `endMeetingConfirmation` strings.
firebase/functions/build.sh	Removes old functions build script.
data_models/build.sh	Removes old data_models build script.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+    test('sets meetingEndedAt and sends email', () async {
+      final event = await createTestEvent();
+      registerFallbackValue(event);
+      registerFallbackValue(EventEmailType.ended);
+


…om the bool check in Settings display

… period.

- Move date into own line below event title - Enlarge event picture and justify left - Add missing label for message participants button - Locate that button with the other event control buttons (cancel and add to calendar), next to the share buttons because it's semantically related - Emphasize that button mainly for aesthetics but also because it's the most common action (cal add is probably only done once, canceling the event is not generally expected, want to encourage messaging through Frankly platform)

Specifically, mini refactor to use it for the calendar_menu_button.

dariusk

This generally looks great! Love to see the debounces and clamps and all that bread and butter logic flow stuff.

I have more specific comments inline, particularly with some instructions on putting in Matomo hooks for analytics.

🫡 Been a pleasure serving with you!!

dariusk · 2026-05-15T20:39:53Z

+              type: ActionButtonType.outline,
+              color: context.theme.colorScheme.surfaceContainerLowest,
+              icon: Icon(
+                CupertinoIcons.calendar_badge_plus,


I thought we were moving away from CupertinoIcons? This was my understanding from working on #223

Yes, this is true. Please use Icons (Material Icons) for consistency!

dariusk · 2026-05-15T20:40:12Z

+      type: ActionButtonType.outline,
+      color: context.theme.colorScheme.surfaceContainerLowest,
+      icon: Icon(
+        CupertinoIcons.paperplane,


See above on CupertinoIcons

dariusk · 2026-05-15T20:40:44Z

+        size: 24,
+        color: context.theme.colorScheme.onSurfaceVariant,
+      ),
+      text: 'Message participants',


Should be a localized string

dariusk · 2026-05-15T20:42:53Z

+                    final endDateTime = scheduledTime.add(
+                      Duration(minutes: eventProvider.event.durationInMinutes),
+                    );
+                    final endFmt = DateFormat('h:mma').format(endDateTime);


Could we possibly reuse some of the time formatting code you add in time_indicator.dart here?

dariusk · 2026-05-15T20:43:20Z

-                    ],
-                  ],
-                ),
+                if (canCancelParticipation || _canEditEvent) ...[                  SizedBox(height: 4),


Strange whitespace needs fixing!

dariusk · 2026-05-15T22:13:09Z

+      }
+      transaction.update(
+        liveMeetingRef,
+        UpdateData.fromMap({


I think right around here we need some hooks for the Matomo API. Basically, wherever we have the call to set the meeting end time (or if it's in two places put it in both places), we should put code close to or identical to what's here:

frankly/client/lib/features/events/features/live_meeting/features/meeting_agenda/data/providers/meeting_agenda_provider.dart

Lines 401 to 410 in 2ac7c13

analytics.logEvent(

AnalyticsCompleteEventEvent(

communityId: communityId,

eventId: eventId,

asHost: (event?.eventType != EventType.hostless) &&

event?.creatorId == userService.currentUserId,

templateId: templateId,

duration: durationInSeconds,

),

);

and remove the linked code from meeting_agenda_provider.dart. The linked code is from an attempt to capture a "Complete Event" event for Matomo analytics. It logs 1) that the Frankly event ended, and 2) how many seconds long the event was. However, the linked code was written back before we really had a concept of "meeting ended" so it's really scattershot and only triggers sometimes. Deleting that code and dropping it in here (with a little math to calculate duration) would be ideal.

If we get this working right we'll be able to see in Matomo a list of anonymized meeting IDs alongside duration of the meeting, like in the following screenshot but with more than just the top entry working correctly

Oh yeah, probably also the hook needs to go in firebase/functions/lib/events/live_meetings/scheduled_end_meeting.dart

dariusk · 2026-05-15T22:15:44Z


+    // On first join, schedule automatic meeting end if the event has
+    // autoEndMeeting enabled, a scheduled time, and a duration.
+    if (result.isFirstJoin) {


This whole conditional branch pleases me to look at

mikewillems added 17 commits April 10, 2026 23:54

Merge branch 'mw/feat/breakout-room-recordings' into mw/feat/event-end

31d044a

Only conflict in client/lib/features/admin/presentation/views/data_tab.dart

merge 'staging' into mw/feat/event-end

3fd465b

Add tests for EndMeetingForAll CF

34cc52d

- Verify meetingEndedAt is written and email sent - Verify idempotency (second call is a no-op) - Verify non-mod/admin/owner is rejected

Schedule automatic meeting end via Cloud Tasks

547212a

- Add ScheduledEndMeeting OnRequestMethod CF with idempotency guard - Add isFirstJoin flag to MeetingJoinResult - Schedule end task on first join when event has scheduledTime + duration - Register ScheduledEndMeeting in main.dart

Add tests for ScheduledEndMeeting CF

c4c3471

- Verify meetingEndedAt is written and email sent - Verify idempotency (second call is a no-op)

Comment _postEventEmailThresholdInMinutes

2910709

Merge branch 'staging' into mw/feat/event-end

b76fd50

# Conflicts: # client/lib/features/admin/presentation/views/data_tab.dart

Add widget tests for VerticalTimeAndDateIndicator end time

4fc78c4

- Verify start-only display - Verify start-end range display

removed defunct old build scripts

e7e4a56

Reject join attempts on ended meetings

5992828

Add test for join rejection on ended meetings

aa1fde9

Prompt host/admin to end meeting on leave

b2ef50a

mikewillems temporarily deployed to staging May 2, 2026 00:24 — with GitHub Actions Inactive

added option to close all rooms at meeting end time, with grace period

62bf5e1

mikewillems temporarily deployed to staging May 2, 2026 00:32 — with GitHub Actions Inactive

mikewillems marked this pull request as ready for review May 2, 2026 00:33

Copilot AI review requested due to automatic review settings May 2, 2026 00:33

Copilot started reviewing on behalf of mikewillems May 2, 2026 00:33 View session

mikewillems marked this pull request as draft May 2, 2026 00:34

Copilot AI reviewed May 2, 2026

View reviewed changes

Fixed settings type error by only filtering listed non-bool values fr…

b42c2bd

…om the bool check in Settings display

mikewillems temporarily deployed to staging May 2, 2026 00:39 — with GitHub Actions Inactive

mikewillems marked this pull request as ready for review May 2, 2026 00:40

mikewillems marked this pull request as draft May 2, 2026 14:44

mikewillems added 10 commits May 3, 2026 14:28

Add grace period input validation and auto-correction

b56e9a0

Extract shared end meeting core logic

e428a3f

Treat dialog dismiss as cancel in host leave prompt

5f89aa0

Parallelize breakout room recording stops with Future.wait

aafef46

Run recording stops in parallel with email delivery

72f7f3e

Use permissionDenied for auth errors

0938e94

Short-circuit auth check for event creator

a978fdc

Guard against double-tap on leave with _leavingInProgress flag

d3b0dcd

Add widget tests for grace period input validation and auto-correction

3b6f104

Added comments explaining input validation for meeting auto-end grace…

cb49d91

… period.

mikewillems force-pushed the mw/feat/event-end branch from e44572e to cb49d91 Compare May 4, 2026 04:10

Merge branch 'staging' into mw/feat/event-end

c98b172

mikewillems temporarily deployed to staging May 4, 2026 04:43 — with GitHub Actions Inactive

mikewillems changed the title ~~Mw/feat/event end~~ Automatic and Manual Event Ending May 4, 2026

mikewillems marked this pull request as ready for review May 4, 2026 04:45

mikewillems temporarily deployed to staging May 4, 2026 16:12 — with GitHub Actions Inactive

Use ActionButton consistently in grouped info card buttons.

49c78aa

Specifically, mini refactor to use it for the calendar_menu_button.

mikewillems temporarily deployed to staging May 4, 2026 16:49 — with GitHub Actions Inactive

Avoid BuildContext use across async gap

9b45973

mikewillems temporarily deployed to staging May 4, 2026 16:57 — with GitHub Actions Inactive

mikewillems requested a review from dariusk May 4, 2026 17:20

Merge branch 'staging' into mw/feat/event-end

4bbfc29

mikewillems temporarily deployed to staging May 5, 2026 17:03 — with GitHub Actions Inactive

Merge branch 'staging' into mw/feat/event-end

6813be8

mikewillems had a problem deploying to staging May 6, 2026 23:44 — with GitHub Actions Failure

regenerated l10n files

f696265

mikewillems temporarily deployed to staging May 7, 2026 00:00 — with GitHub Actions Inactive

dariusk suggested changes May 15, 2026

View reviewed changes

test

8a975ec

	analytics.logEvent(
	AnalyticsCompleteEventEvent(
	communityId: communityId,
	eventId: eventId,
	asHost: (event?.eventType != EventType.hostless) &&
	event?.creatorId == userService.currentUserId,
	templateId: templateId,
	duration: durationInSeconds,
	),
	);

Conversation

mikewillems commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Automatic and Manual Event Ending

What is in this PR?

Changes in the codebase

General Approach

Non-obvious choices

Changes by Commit

Changes outside the codebase

Testing this PR

Additional information

Uh oh!

github-actions Bot commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dariusk left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

katherineqian May 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

mikewillems commented May 2, 2026 •

edited

Loading

github-actions Bot commented May 2, 2026 •

edited

Loading

katherineqian May 18, 2026 •

edited

Loading