Skip to content
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

[Fix]Ensure Call.state.startedAt is non-nil when session is valid #493

Merged

Conversation

ipavlidakis
Copy link
Collaborator

@ipavlidakis ipavlidakis commented Aug 22, 2024

📝 Summary

This revision aims to resolve an issue where callDuration may not be populated from the SFU.

🧪 Manual Testing Notes

Join and leave multiple calls. The call duration on top of the call should always be visible after joining a call.

☑️ Contributor Checklist

  • I have signed the Stream CLA (required)
  • This change follows zero ⚠️ policy (required)
  • This change should receive manual QA
  • Changelog is updated with client-facing changes
  • New code is covered by unit tests
  • Comparison screenshots added for visual changes
  • Affected documentation updated (Docusaurus, tutorial, CMS)

@ipavlidakis ipavlidakis added the bug Something isn't working label Aug 22, 2024
@ipavlidakis ipavlidakis self-assigned this Aug 22, 2024
@ipavlidakis ipavlidakis requested a review from a team as a code owner August 22, 2024 14:29
@Stream-SDK-Bot
Copy link
Collaborator

SDK Size

title develop branch diff status
StreamVideo 7.31 MB 7.31 MB 0 KB 🟢
StreamVideoSwiftUI 2.18 MB 2.18 MB 0 KB 🟢
StreamVideoUIKit 2.32 MB 2.32 MB 0 KB 🟢
StreamWebRTC 8.69 MB 8.69 MB 0 KB 🟢

Copy link

1 Warning
⚠️ Please be sure to complete the Contributor Checklist in the Pull Request description

Generated by 🚫 Danger

Copy link

sonarcloud bot commented Aug 22, 2024

@ipavlidakis ipavlidakis enabled auto-merge (squash) August 22, 2024 15:05
@ipavlidakis ipavlidakis merged commit 03c3a30 into develop Aug 22, 2024
13 checks passed
@ipavlidakis ipavlidakis deleted the fix/call-state-startedAt-is-not-nil-when-session-is-valid branch August 22, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants