1333905 - Documentation for browser.notification.getAll() is incorrect

Status: RESOLVED FIXED

(Developer Documentation Graveyard :: Add-ons, defect)

Description

[e.chechurin]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.67 Safari/537.36

Steps to reproduce:

Used a notification API like so:

browser.notification.getAll().then(function (results) {
  console.log(results)
})



Actual results:

`results` contains a chrome-like object value.


Expected results:

`results` should contain "... an array of strings, each of which is the ID of a currently active notification."
source: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/notifications/getAll

Comment 1

[e.chechurin]

might be relative:
https://bugzilla.mozilla.org/show_bug.cgi?id=1254359

Comment 2

[Lina Butler (ex-Mozilla)]

Bug 1254359 definitely looks related. Are you on Nightly, Dev Edition, Beta, or Release?

Comment 3

[Will Bamberg [:wbamberg]]

Thanks for filing this. I've updated the page. Bob, would you mind checking (especially "Return value" and the examples).
https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/notifications/getAll

Comment 4

[e.chechurin]

You welcome.

Guys, on a separate note - do you have plan to add Typescript typings for 'browser'? 
We have covered a bunch of stuff in our project (based on chrome typings), and I thought maybe we can help with 'official' typings?

Comment 5

[Will Bamberg [:wbamberg]]

(In reply to e.chechurin from comment #4)
> You welcome.
> 
> Guys, on a separate note - do you have plan to add Typescript typings for
> 'browser'? 

That's interesting! I don't know, personally, but I suggest you'll get better responses if you ask on the dev-addons mailing list: https://mail.mozilla.org/listinfo/dev-addons.

Comment 6

[Bob Silverberg [:bsilverberg]]

Looks great, thanks Will!