Closed
Bug 1019733
Opened 12 years ago
Closed 12 years ago
Point the client code to the right url
Categories
(Hello (Loop) :: Client, defect, P2)
Hello (Loop)
Client
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla33
People
(Reporter: tarek, Assigned: standard8)
References
Details
(Whiteboard: [p=0])
Attachments
(1 file, 2 obsolete files)
|
6.11 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
once the prod is deployed, the client side needs to point to https://loop.services.mozilla.com
| Reporter | ||
Updated•12 years ago
|
Assignee: nobody → standard8
| Reporter | ||
Comment 1•12 years ago
|
||
I have no idea what's the product/component - or if there's already a tracking bug for this. I just want to make sure we're coordinating on the loop server final url once it's deployed
Flags: needinfo?(standard8)
| Assignee | ||
Comment 3•12 years ago
|
||
Loop/Client is what we're using for now.
Component: General → Client
Flags: needinfo?(standard8)
OS: Mac OS X → All
Priority: -- → P2
Product: Mozilla Services → Loop
Hardware: x86 → All
Whiteboard: [qa+] → [qa+][p=0.25]
Target Milestone: --- → mozilla32
| Assignee | ||
Comment 4•12 years ago
|
||
Here's the alteration to the preference file for the production server, ready for when we need it. I'll request review once it is fully up and running.
Alexis - for now please check I've got it right.
Attachment #8434161 -
Flags: feedback?(alexis+bugs)
| Assignee | ||
Updated•12 years ago
|
Whiteboard: [qa+][p=0.25] → [qa+][p=0]
Comment 5•12 years ago
|
||
Comment on attachment 8434161 [details] [diff] [review]
Update loop.server preference to the production server
Seems correct. Not sure about the trailing slash though.
Attachment #8434161 -
Flags: feedback?(alexis+bugs) → feedback+
| Assignee | ||
Comment 6•12 years ago
|
||
Now without the trailing slash.
Attachment #8434161 -
Attachment is obsolete: true
| Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 8434900 [details] [diff] [review]
Update loop.server preference to the production server v2
Adam, the server isn't ready yet, but I'll hold this in my queue until we get clearance and we've tested it. Getting review early so it is ready when we need it.
Attachment #8434900 -
Flags: review?(adam)
Comment 8•12 years ago
|
||
Comment on attachment 8434900 [details] [diff] [review]
Update loop.server preference to the production server v2
Review of attachment 8434900 [details] [diff] [review]:
-----------------------------------------------------------------
r+ with nit.
::: browser/app/profile/firefox.js
@@ +1502,5 @@
> pref("image.mem.max_decoded_image_kb", 256000);
>
> #ifdef MOZ_LOOP
> // XXX This is a temporary development server. The production one will be
> // coming soon.
Remove this comment.
Attachment #8434900 -
Flags: review?(adam) → review+
| Assignee | ||
Comment 9•12 years ago
|
||
Removed the comment, ready for landing when we get the go ahead.
Attachment #8434900 -
Attachment is obsolete: true
Attachment #8435049 -
Flags: review+
| Assignee | ||
Comment 10•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 11•12 years ago
|
||
| Assignee | ||
Updated•12 years ago
|
Target Milestone: mozilla32 → mozilla33
Comment 12•12 years ago
|
||
Untracking for QA. Please needinfo me to request testing.
Flags: qe-verify-
Whiteboard: [qa+][p=0] → [p=0]
You need to log in
before you can comment on or make changes to this bug.
Description
•