feat(meeting-form): fellowship prefix on external Zoom/Calendar titles - #563
Conversation
External titles now read "{Fellowship} {Title} - {Mode}": AA/Al-Anon derive from the
Categories checkboxes, and checking Other reveals an optional custom fellowship name
stored in a new nullable Meeting.fellowship column. A pinned zoomTopic still wins.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
@coderabbitai summary
Description
Meeting.fellowshipcolumn holding only the custom text typed behind the "Other" category; AA/Al-Anon are never stored redundantly — they derive fromcalTypeat title-build time.fellowshipPrefixedTitle()helper — external names now read{Fellowship} {Title} - {Mode + days}(e.g.AA/Al-Anon Serenity Now - Hybrid Mon-Fri), fixed AA → Al-Anon → custom order,/-joined; no fellowship → title passes through unchanged.zoomTopicstill short-circuits first, so adopted legacy Zoom names are never renamed.fellowshipdeclared in the zod schema (it strips undeclared keys) and added to the scoped-edit child and linked-sibling copy lists.Testing
cd frontend && yarn test:unit tests/unit/linkedSchedules.test.ts tests/unit/zoom.test.ts tests/unit/googleCalendar.test.tsyarn test:component tests/component/useMeetingForm.test.tsx tests/component/NewMeeting.test.tsxyarn test:integration tests/integration/write-meeting-route.test.ts tests/integration/update-meeting-scoped-edit.test.tsAA/<custom>prefix.Area(s) Touched
Product areas
Integrations
Engineering
Pre-merge Checklist
yarn lint).