Closed
Bug 750789
Opened 12 years ago
Closed 12 years ago
Hide YouSendIt graph / stats if they don't finish their API in time
Categories
(Thunderbird :: Preferences, defect)
Tracking
(thunderbird13+ fixed, thunderbird14- unaffected)
RESOLVED
FIXED
Thunderbird 13.0
Tracking | Status | |
---|---|---|
thunderbird13 | + | fixed |
thunderbird14 | - | unaffected |
People
(Reporter: mconley, Assigned: mconley)
References
Details
Attachments
(5 files)
4.10 KB,
patch
|
andreasn
:
review+
bwinton
:
ui-review+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
50.05 KB,
image/png
|
Details | |
59.09 KB,
image/png
|
Details | |
70.15 KB,
image/png
|
Details | |
28.92 KB,
image/png
|
Details |
The YSI v2 API gives us the amount of storage space that a user has been allocated, but does not give us any information about how much storage space a user has used. YSI is working to fix this. In the event that they can not finish it in time, we should have a fallback for the YSI management pane. Blake and I discussed this, and we thought that simply displaying the YSI settings link in the center of the pane would be sufficient.
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 years ago
|
||
Assignee | ||
Comment 4•12 years ago
|
||
Assignee | ||
Comment 5•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Attachment #619964 -
Flags: ui-review?(bwinton)
Attachment #619964 -
Flags: review?(nisses.mail)
Comment 6•12 years ago
|
||
Comment on attachment 619964 [details] [diff] [review] Patch v1 Looks good to me. Thanks for the screenshots!
Attachment #619964 -
Flags: ui-review?(bwinton) → ui-review+
Comment 7•12 years ago
|
||
Comment on attachment 619964 [details] [diff] [review] Patch v1 Too bad about more px values in this code, but not much we can do about that (from poking around with DOMi for a couple of minutes) it seems.
Attachment #619964 -
Flags: review?(nisses.mail) → review+
Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 619964 [details] [diff] [review] Patch v1 David has been in communication with the YSI folks, and they don't think they can provide the values in time for TB 13, and have opted to use this placeholder instead.
Attachment #619964 -
Flags: approval-comm-beta?
Assignee | ||
Updated•12 years ago
|
tracking-thunderbird13:
--- → ?
Updated•12 years ago
|
Attachment #619964 -
Flags: approval-comm-beta? → approval-comm-beta+
Assignee | ||
Comment 9•12 years ago
|
||
comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/c1fb8dc6f817
Status: NEW → RESOLVED
Closed: 12 years ago
status-thunderbird13:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
Updated•12 years ago
|
Assignee | ||
Comment 10•12 years ago
|
||
David: Do you know if YSI has finished their API for reporting storage usage stats yet? If not, we'll need to land this on beta... -Mike
tracking-thunderbird14:
--- → ?
Comment 11•12 years ago
|
||
(In reply to Mike Conley (:mconley) from comment #10) > David: > > Do you know if YSI has finished their API for reporting storage usage stats > yet? I don't know. I know they've got a large patch coming to work with their v3 api, and it wouldn't surprise me if we had to use that api. I guess we've got a few weeks to decide.
Updated•12 years ago
|
Comment 12•12 years ago
|
||
the status UI does work with the v3 api patch - so the question is whether the useful parts of that can land in beta, i.e., can the patch work without string changes?
Assignee | ||
Comment 13•12 years ago
|
||
Where is the v3 patch?
Comment 14•12 years ago
|
||
bug 763008
Comment 15•12 years ago
|
||
(In reply to David :Bienvenu from comment #12) > the status UI does work with the v3 api patch - so the question is whether > the useful parts of that can land in beta, i.e., can the patch work without > string changes? It looks very much like the v3 patch will land for beta.
Comment 16•12 years ago
|
||
v3 has now landed in beta for TB 14, so this is not necessary there.
status-thunderbird14:
--- → unaffected
You need to log in
before you can comment on or make changes to this bug.
Description
•