Closed Bug 1510988 Opened 2 years ago Closed 2 years ago

Possible improvements for the new telemetry snapshot APIs


(Toolkit :: Telemetry, enhancement, P1)




Tracking Status
firefox66 --- fixed


(Reporter: standard8, Assigned: janerik)




(3 files)

I've just come across these as I'm working on a patch that was using the old APIs. 

There's a few things that I don't understand with the new APIs. It could be that I don't have enough context here to know the reasoning behind them, if so, please enlighten me.

- The aClearStore parameter seems like it could be made optional. The old version of the API had it default to false, and I don't see why it couldn't be made to default false here as well.

- I find the documentation for the aStoreName parameter strange: 'Something like "main"' doesn't really tell me what it is about or where I would find out what names to actually use. After digging, I found a comment in that stores are still a WIP.

Maybe this could be changed to 'Use "main" for now' or '@see <url>' (with a bit of documentation behind it)?

- I don't know the future of stores, or what they will be, but could the aStoreName parameter also default to "main"?
Assignee: nobody → jrediger
Blocks: 1498159
Priority: -- → P2
Thanks for reporting. We are indeed lacking some better documentation currently, but plan to add that in the follow-ups after mozlando.
I take the other suggestions into consideration as well.
Priority: P2 → P3
Priority: P3 → P2
Priority: P2 → P1
Pushed by
Make the clear flag optional in snapshot API r=chutten
Clearly document what the store name in snapshot APIs is for r=chutten
Make store name optional and default to "main" r=chutten
You need to log in before you can comment on or make changes to this bug.