Skip to content

Add missing fields to the update conversation endpoint#150

Merged
cbj0hns0n merged 10 commits into
mainfrom
cj/edit-event
Jun 11, 2026
Merged

Add missing fields to the update conversation endpoint#150
cbj0hns0n merged 10 commits into
mainfrom
cj/edit-event

Conversation

@cbj0hns0n

Copy link
Copy Markdown
Contributor

What is in this PR?

The update conversation endpoint only supported a handful of fields. This adds everything the edit event form needs: scheduled times, platforms, features, topic, and conversation type.

Changes in the codebase

  • Expanded the update endpoint's validation to accept scheduledTime, scheduledEndTime, topicId, type, platforms, properties, and features
  • Properties now merge with existing values instead of overwriting, so only changed keys are affected; switching conversation type also recreates the agents and adapters for that type
  • conversationType is no longer marked immutable, so it can be updated on an existing event
  • Added alternateName to speakers and moderators in both the create and update schemas

Documentation and automated testing

Did you:

  • document any breaking changes in your commit messages?
  • document your changes as comments in the code? Use TSDoc format where appropriate.
  • update the README and docs to be clear and easy to use for end users and developers?
  • add and/or update automated tests?
  • update team documentation of any new or changed environment variables?
  • add a What's New entry in content/whatsNew.ts for new features?

Testing this PR

Related to berkmancenter/nextspace#127, test steps are listed there.

@cbj0hns0n cbj0hns0n force-pushed the cj/edit-event branch 2 times, most recently from ab5c907 to 129fbd7 Compare June 10, 2026 23:58
@cbj0hns0n cbj0hns0n merged commit 2db5355 into main Jun 11, 2026
1 check passed
@cbj0hns0n cbj0hns0n deleted the cj/edit-event branch June 11, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant