1450998 - Improve API and docs for Remote Settings

Status: RESOLVED FIXED

(Firefox :: Remote Settings Client, enhancement)

Description

[Mathieu Leplatre [:leplatrem]]

Since 1397230, we have a generic implementation of remote settings.

We should:

- add a get() method
- have RemoteSettings.pollChanges() instead of a standalone function
- add usage documentation

Comment 1

[Mathieu Leplatre [:leplatrem]]

Attachment: Bug 1450998 - Improve API and docs for Remote Settings

Review commit: https://reviewboard.mozilla.org/r/233286/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233286/

Comment 2

[Mathieu Leplatre [:leplatrem]]

Attachment: Bug 1450998: Part 2: Introduce RemoteSettings("key").get()

Review commit: https://reviewboard.mozilla.org/r/233288/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233288/

Comment 3

[Mathieu Leplatre [:leplatrem]]

Attachment: Bug 1450998: Part 3: Use bucket "main" by default

Review commit: https://reviewboard.mozilla.org/r/233290/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233290/

Comment 4

[Mathieu Leplatre [:leplatrem]]

Comment on attachment 8964577 [details]
Bug 1450998: Part 2: Introduce RemoteSettings("key").get()

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233288/diff/1-2/

Comment 5

[Mathieu Leplatre [:leplatrem]]

Comment on attachment 8964578 [details]
Bug 1450998: Part 3: Use bucket "main" by default

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233290/diff/1-2/

Comment 6

[Mark Goodwin [:mgoodwin]]

Comment on attachment 8964576 [details]
Bug 1450998 - Improve API and docs for Remote Settings

https://reviewboard.mozilla.org/r/233286/#review240084

Looks good to me. Thanks.

Comment 7

[Mark Goodwin [:mgoodwin]]

Comment on attachment 8964577 [details]
Bug 1450998: Part 2: Introduce RemoteSettings("key").get()

https://reviewboard.mozilla.org/r/233288/#review240088

Comment 8

[Mark Goodwin [:mgoodwin]]

Comment on attachment 8964578 [details]
Bug 1450998: Part 3: Use bucket "main" by default

https://reviewboard.mozilla.org/r/233290/#review240090

Comment 9

[Mathieu Leplatre [:leplatrem]]

Comment on attachment 8964576 [details]
Bug 1450998 - Improve API and docs for Remote Settings

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233286/diff/1-2/

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eed22d09bb9a
Improve API and docs for Remote Settings r=mgoodwin

Comment 11

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/eed22d09bb9a