Avatar support + remove krisp #14
Conversation
| @@ -27,6 +27,13 @@ struct ChatView: View { | |||
| } | |||
| .listStyle(.plain) | |||
| .scrollIndicators(.hidden) | |||
| .mask( | |||
| @EnvironmentObject private var room: Room | ||
| @EnvironmentObject private var participant: Participant | ||
|
|
||
| private var worker: RemoteParticipant? { |
There was a problem hiding this comment.
Shall we move it to Room extension for a clearer picture?
There was a problem hiding this comment.
it would need to move to a new Participant extension if anything, but our focus should be on just solving this in the SDK automatically.
| SwiftUIVideoView(cameraTrack) | ||
| .clipShape(RoundedRectangle(cornerRadius: 8)) | ||
| } else { | ||
| AgentBarAudioVisualizer(audioTrack: micTrack, agentState: participant.agentState, barColor: .primary, barCount: 5) |
There was a problem hiding this comment.
I see some opportunity to merge it with ParticipantView with very similar logic.
There was a problem hiding this comment.
yeah participantview needs to be updated to handle agent state. ideally once we make the necessary framework updates this whole sample app becomes quite simple
|
@pblazej weird I don't know how that even happened. good catch. i had a scheme and it was marked shared but i had to delete and recreate it for the xcshareddata entry to appear |
This started as a few tweaks to #12 but I needed to make some deeper changes to reliably load the avatar and also incorporated design direction from discussion with @mattherzog (although it's not final and isn't based on a specific mockup)
I also removed krisp while I was in here, in favor of the agent-side krisp