-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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 not sending CDC message when passcode is cancelled on TV app #34507
Fix not sending CDC message when passcode is cancelled on TV app #34507
Conversation
PR #34507: Size comparison from 1b83c3a to b47e947 Full report (8 builds for cc32xx, mbed, qpg, stm32, tizen)
|
PR #34507: Size comparison from 1b1340f to 2998228 Full report (20 builds for cc13x4_26x4, cc32xx, mbed, nrfconnect, nxp, qpg, stm32, tizen)
|
PR #34507: Size comparison from bfa80b9 to 0033a12 Full report (3 builds for cc32xx, stm32)
|
PR #34507: Size comparison from c91a779 to bf1679e Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 42d03d5 to 9bfc296 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 9ef5bbf to 1b882bc Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 4334e91 to be51916 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 95bf668 to 9bd23cd Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 5e3127f to e36a059 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from d9bcdd3 to 4cebfc2 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 26c816a to 95492a0 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34507: Size comparison from 157aa40 to e8335e7 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Problem
After customers cancel passcode entry on TV app, CDC message is not sent to the tv-casting-app
Change overview
Test on TV Linux App - High level
Test - Steps:
Build the TV app using command:
scripts/examples/gn_build_example.sh examples/tv-app/linux out/debug/
Build the TV casting app using command:
scripts/examples/gn_build_example.sh examples/tv-casting-app/linux out/debug/
Start TV App
out/debug/chip-tv-app
Start TV Casting App
out/debug/chip-tv-casting-app
On TV App: Uninstall the app
app uninstall 65521 32769
On TV Casting App: Send casting request to the tv-app using
cast setusecommissionerpasscode 1
On TV Casting App: Send casting request to the tv-app using
cast request 0
On TV App: Accepted the Cast request from the tv-casting app
controller ux ok
On TV App: Cancel PIN code
controller ux cancel
Test - Logs:
Passcode cancelled: true
Passcode cancelled: true