Closed Bug 1450998 Opened 2 years ago Closed 2 years ago

Improve API and docs for Remote Settings

Categories

(Firefox :: Remote Settings Client, enhancement)

61 Branch
enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: leplatrem, Assigned: leplatrem)

References

Details

Attachments

(1 file, 2 obsolete files)

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
Attachment #8964576 - Flags: review?(mgoodwin)
Attachment #8964577 - Flags: review?(mgoodwin)
Attachment #8964578 - Flags: review?(mgoodwin)
Assignee: nobody → mathieu
Blocks: 1451031
Blocks: 1451035
Blocks: 1451050
Version: 57 Branch → 61 Branch
Component: Blocklisting → Remote Settings Client
Product: Toolkit → Firefox
Target Milestone: --- → Firefox 61
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.
Attachment #8964576 - Flags: review?(mgoodwin) → review+
Comment on attachment 8964577 [details]
Bug 1450998: Part 2: Introduce RemoteSettings("key").get()

https://reviewboard.mozilla.org/r/233288/#review240088
Attachment #8964577 - Flags: review?(mgoodwin) → review+
Comment on attachment 8964578 [details]
Bug 1450998: Part 3: Use bucket "main" by default

https://reviewboard.mozilla.org/r/233290/#review240090
Attachment #8964578 - Flags: review?(mgoodwin) → review+
Attachment #8964577 - Attachment is obsolete: true
Attachment #8964578 - Attachment is obsolete: true
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/eed22d09bb9a
Improve API and docs for Remote Settings r=mgoodwin
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/eed22d09bb9a
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.