Closed
Bug 1029947
Opened 10 years ago
Closed 10 years ago
[B2G][NFC] demo app to support bug 979152
Categories
(Firefox OS Graveyard :: NFC, defect)
Tracking
(feature-b2g:2.1, tracking-b2g:backlog, b2g-v2.1 fixed)
RESOLVED
FIXED
2.1 S3 (29aug)
Tracking | Status | |
---|---|---|
b2g-v2.1 | --- | fixed |
People
(Reporter: wesley_huang, Assigned: tauzen)
References
Details
User Story
Scope: the demo app is for showing HCI event/notifications
Attachments
(1 file)
No description provided.
Reporter | ||
Updated•10 years ago
|
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → kmioduszewski
Reporter | ||
Updated•10 years ago
|
Reporter | ||
Updated•10 years ago
|
User Story: (updated)
Updated•10 years ago
|
QA Whiteboard: [COM=NFC]
Assignee | ||
Comment 1•10 years ago
|
||
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
Reporter | ||
Comment 2•10 years ago
|
||
Hi Krzysztof,
Can you set the target milestone to either sprint2 or sprint3?
Assignee | ||
Comment 3•10 years ago
|
||
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)
Reporter | ||
Comment 4•10 years ago
|
||
Maybe Alive can help to suggest. We do need to place the demo app somewhere (engineering build), but not necessary in user build.
Flags: needinfo?(alive)
Comment 5•10 years ago
|
||
Yes, dev_apps is where you could place any test apps.
Flags: needinfo?(alive)
Comment 6•10 years ago
|
||
Hi, do you have any update for this bug? We need to land this bug before 8/29. Thanks.
Flags: needinfo?(kmioduszewski)
Assignee | ||
Comment 7•10 years ago
|
||
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?
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(kmioduszewski)
Assignee | ||
Comment 8•10 years ago
|
||
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?
Flags: needinfo?(timdream)
Comment 9•10 years ago
|
||
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.
Flags: needinfo?(timdream)
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8476612 [details]
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)
Updated•10 years ago
|
Attachment #8476612 -
Flags: review?(timdream) → review+
Updated•10 years ago
|
QA Whiteboard: [COM=NFC]
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 11•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v2.1:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Updated•10 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
You need to log in
before you can comment on or make changes to this bug.
Description
•