Scope: the demo app is for showing HCI event/notifications
No description provided.
Assignee: nobody → kmioduszewski
WIP demo app source code is here: https://github.com/tauzen/HCI-Event-Demo. This version uses obsolete Gecko patch from Bug 979767 to get the HCI EVT_TRANSACTION notification. There is still an ongoing discussion about how to deliver the notification to Gaia app, see here Bug 1037380 and here https://groups.google.com/d/topic/mozilla.dev.webapi/oER2OLg40pg/discussion. Once this will be agreed and implemented in gecko, the app will be updated.
Depends on: 979767
Hi Krzysztof, Can you set the target milestone to either sprint2 or sprint3?
Done. As this is a test app I'm not sure where in gaia should I place this. Should I add this to dev_apps folder?
Depends on: 1037380
Target Milestone: --- → 2.1 S3 (29aug)
Maybe Alive can help to suggest. We do need to place the demo app somewhere (engineering build), but not necessary in user build.
Yes, dev_apps is where you could place any test apps.
Hi, do you have any update for this bug? We need to land this bug before 8/29. Thanks.
Assuming that there won't be much changes introduced in Bug 979767, this could be landed. It's working for me with the latest code from Garners development branch. Who could review this?
Hello Tim, This is a demo app for testing user story from Bug 979152. It will be used by QA so I want to land this in gaia dev_apps. Could you point out a person who could review this?
I can rubberstamp this. We actually against putting things as a new app (API manual tests/demo should be put in UI Test app), however since this app needs to handle system message, we might as well put one more app into dev_apps.
Comment on attachment 8476612 [details] [review] pull-request-1029947.txt Hi Tim, since the decision in Bug 979767 was made to use only system-message it would be good if you could review this. Changes to the app should not be needed, we might require to change one property name in the manifest file.
Attachment #8476612 - Flags: review?(timdream)
Attachment #8476612 - Flags: review?(timdream) → review+
You need to log in before you can comment on or make changes to this bug.