Closed Bug 1034533 Opened 5 years ago Closed 5 years ago

[NFC] Wallet demo app for testing Secure Element API

Categories

(Firefox OS Graveyard :: NFC, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(feature-b2g:2.2+, tracking-b2g:backlog)

RESOLVED FIXED
2.2 S1 (5dec)
feature-b2g 2.2+
tracking-b2g backlog

People

(Reporter: tauzen, Assigned: tauzen)

References

Details

(Whiteboard: [ETA: 12/15])

Attachments

(1 file, 1 obsolete file)

Once we will have Secure Element API in place we will need a demo wallet app for testing the new API and showcasing OS capabilities.
I might re-use the old wallet implementation available here: https://github.com/lantianyouyou/mwallet
Assignee: nobody → kmioduszewski
Blocks: 979158, b2g-nfc
Depends on: 879861
Whiteboard: [ETA: 12/15]
blocking-b2g: --- → backlog
feature-b2g: --- → 2.2+
Target Milestone: --- → 2.2 S2 (19dec)
Attached file pull-request-1034533.txt (obsolete) —
Wallet demo using the implementation of SE API (Bug 879861). Current functionalities:
- listing payment applets
- activation/deactivation of payment applet on the contact less interface
Attachment #8529192 - Attachment mime type: text/plain → text/x-github-pull-request
Comment on attachment 8529192 [details] [review]
pull-request-1034533.txt

Hello Tim, as in the case of Bug 1029947, we would also like to put this app in dev_apps folder, since this showcases API usage and will also be used by QA to perform testing. Could you review this app?
Attachment #8529192 - Flags: review?(timdream)
Comment on attachment 8529192 [details] [review]
pull-request-1034533.txt

For the purpose of testing you put the UIs in "UI Tests" app would be enough. To demo/showcase the work you don't really need to check in an entire app :)

Please note that we really want to keep the build size of eng build small. Thanks!
Attachment #8529192 - Flags: review?(timdream)
As requested, I've refactored Wallet Demo code (removed UI code and resources, simplified app logic) and moved it to new section of "UI tests - Privileged App".

I liked better the original app ;), but I understand your concerns. Could you review this version?
Attachment #8529192 - Attachment is obsolete: true
Attachment #8529890 - Flags: review?(timdream)
Attachment #8529890 - Flags: review?(timdream) → review+
Thanks! Gaia-try is green.
Keywords: checkin-needed
https://github.com/mozilla-b2g/gaia/commit/7695e18be7b6bb62b01c17945f1963ca15aa1690
Status: NEW → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: 2.2 S2 (19dec) → 2.2 S1 (5dec)
Blocks: 1133506
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.