Closed
Bug 1034533
Opened 10 years ago
Closed 10 years ago
[NFC] Wallet demo app for testing Secure Element API
Categories
(Firefox OS Graveyard :: NFC, defect)
Tracking
(feature-b2g:2.2+, tracking-b2g:backlog)
RESOLVED
FIXED
2.2 S1 (5dec)
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.
Assignee | ||
Comment 1•10 years ago
|
||
I might re-use the old wallet implementation available here: https://github.com/lantianyouyou/mwallet
Updated•10 years ago
|
Whiteboard: [ETA: 12/15]
Updated•10 years ago
|
blocking-b2g: --- → backlog
feature-b2g: --- → 2.2+
Updated•10 years ago
|
Target Milestone: --- → 2.2 S2 (19dec)
Assignee | ||
Comment 2•10 years ago
|
||
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
Assignee | ||
Updated•10 years ago
|
Attachment #8529192 -
Attachment mime type: text/plain → text/x-github-pull-request
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8529192 [details]
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 4•10 years ago
|
||
Comment on attachment 8529192 [details]
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)
Assignee | ||
Comment 5•10 years ago
|
||
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)
Updated•10 years ago
|
Attachment #8529890 -
Flags: review?(timdream) → review+
Comment 7•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: 2.2 S2 (19dec) → 2.2 S1 (5dec)
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
•