Closed Bug 610607 Opened 9 years ago Closed 9 years ago
suggested Password Manager API implementation
I have mistakenly put this into bug 568937, but still I believe it could be worthy not to have blank space next to Password Manager on https://wiki.mozilla.org/Labs/Jetpack/SDK/APIs
Awesome, thanks for posting this! Next step would be to put up some code examples and send to the mailing list for API feedback. I added this to the SDK APIs page. While we all agreed this is key for 1.0, it's not yet slotted into a release AFAICT.
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product. To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Version: Trunk → unspecified
Assignee: nobody → rFobic
Please note that API review was done under the Bug 637292.
Attachment #519145 - Flags: review?(myk)
Comment on attachment 519145 [details] Pointer to pull request Dietrich: can you take on this review as well?
Attachment #519145 - Flags: review?(myk) → review?(dietrich)
Yep. Irakli, are Myk's comments addressed in the latest version?
Yes I believe they are
Dietrich I have updated pull request to address your comments made there.
Comment on attachment 519145 [details] Pointer to pull request r+ during the irc+pull-request review process.
Attachment #519145 - Flags: review?(dietrich) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.