Skip to content

fix: add null checks for call in getGrid and updateVideoGrid functions [WPB-25267]#21240

Merged
screendriver merged 2 commits into
masterfrom
fix/null-check-for-video-grid-handler
May 6, 2026
Merged

fix: add null checks for call in getGrid and updateVideoGrid functions [WPB-25267]#21240
screendriver merged 2 commits into
masterfrom
fix/null-check-for-video-grid-handler

Conversation

@zskhan
Copy link
Copy Markdown
Contributor

@zskhan zskhan commented May 5, 2026

BugWPB-25267 [Web] Call Ending with error

Add null checks for call in getGrid and updateVideoGrid.
Previously it was throwing exception when the call is undefined.

thisisamir98
thisisamir98 previously approved these changes May 5, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 15
  • Failed: 0
  • Skipped: 0
  • 🔁 Flaky: 0
  • 📊 Total: 15
  • Total Runtime: 104.3s (~ 1 min 44 sec)

Comment thread apps/webapp/src/script/repositories/calling/videoGridHandler.ts Outdated
Co-authored-by: Christian Rackerseder <git@echooff.de>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

Comment thread apps/webapp/src/script/repositories/calling/videoGridHandler.ts
@zskhan zskhan added this pull request to the merge queue May 6, 2026
@screendriver screendriver removed this pull request from the merge queue due to a manual request May 6, 2026
@screendriver screendriver added this pull request to the merge queue May 6, 2026
Merged via the queue into master with commit 181edbb May 6, 2026
35 checks passed
@screendriver screendriver deleted the fix/null-check-for-video-grid-handler branch May 6, 2026 09:08
github-merge-queue Bot pushed a commit that referenced this pull request May 6, 2026
* chore: Update translations

* chore: use github hoster runners for jobs (#20910)

* chore: trigger push

* chore(E2E): Updated calling service API client to use 1Password [WPB-24489] (#20905)

* chore: Update translations

* chore: Update translations

* chore: Revert "fix: Search results are not fetching all files when the search box is empty(WPB-20817) (#21066)"

chore: Revert "fix: Search results are not fetching all files when the search box is  empty(WPB-20817) (#21066)"

* chore: Update translations

chore: Update translations

* chore: Update translations

* fix: add null checks for call in getGrid and updateVideoGrid functions

* Apply suggestion from @screendriver

Co-authored-by: Christian Rackerseder <git@echooff.de>

---------

Co-authored-by: Amir Ghezelbash <thisisamir98@gmail.com>
Co-authored-by: Immad Abdul Jabbar <immad.jabbar@wire.com>
Co-authored-by: Ivan Skvortsov <ivan.skvortsov@wire.com>
Co-authored-by: Christian Rackerseder <git@echooff.de>
Co-authored-by: Zafar Saeed Khan <zafarsaeed@outlook.com>
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.

4 participants