Skip to content

feat: Add support for participant name, metadata, and attributes #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

MahmoudHemaid
Copy link
Contributor

Summary

This PR introduces support for additional participant information and improves the token generation process:

  • Adds a new input for participant_name, separate from participant_id, to allow specifying a display name for participants.
  • Adds support for custom metadata (string) and attributes (key-value pairs) for participants.
  • Introduces a new AttributesEditor component for managing participant attributes in the UI.
  • Changes the /api/token endpoint from GET to POST and updates it to accept the new fields in the request body.
  • Updates user settings and configuration to include participant_name, metadata, and attributes.
  • Refactors the connection logic to send the new fields when requesting a token.
  • Updates the backend API to handle the new POST request and fields.
  • Ensures backward compatibility by defaulting to generated values if fields are not provided.

Screenshot

image

Copy link

vercel bot commented May 25, 2025

@MahmoudHemaid is attempting to deploy a commit to the long-livekitio's projects Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented May 25, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@bcherry bcherry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is great! I've got a couple nitpicks on the UX but I'll just merge as-is and address them in #144

@bcherry bcherry merged commit 9e2b7fc into livekit:main May 28, 2025
1 of 3 checks passed
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.

3 participants