Closed
Bug 1217002
Opened 10 years ago
Closed 10 years ago
Deploy splice 1.1.35 to stage
Categories
(Content Services Graveyard :: Tiles: Ops, defect)
Content Services Graveyard
Tiles: Ops
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rrosario, Unassigned)
Details
Attachments
(1 file)
|
12.73 KB,
application/json
|
Details |
These are only UI changes. No python or DB changes.
Thanks!
Comment 2•10 years ago
|
||
I will need some instructions on what to do and how to do it.
Is there anything on a wiki somewhere?
Also I don't have access to datadog
Flags: needinfo?(mschifer)
Comment 3•10 years ago
|
||
:r1cky it's deployed in stage, but the UI changes seem have some issues:
The drop-down for re-deploying previous distributions is gone.
Images don't appear in the preview when using the attached payload.
Flags: needinfo?(rrosario)
| Reporter | ||
Comment 4•10 years ago
|
||
(In reply to Daniel Thorn [:relud] from comment #3)
> The drop-down for re-deploying previous distributions is gone.
Yeah, we removed that removed that feature because it isn't really used.
> Images don't appear in the preview when using the attached payload.
The problem with the tile images in that test file is they are white with a transparent background. I was able to see by hacking the CSS of the page. We should probably create a better file. Let me know if you need help with that.
Flags: needinfo?(rrosario)
Comment 5•10 years ago
|
||
If you upload android.json in the old UI the images appear to include background colors (blue, green, orange, and red). Not sure what the difference is, but something definitely changed there.
| Reporter | ||
Comment 6•10 years ago
|
||
(In reply to Daniel Thorn [:relud] from comment #5)
> If you upload android.json in the old UI the images appear to include
> background colors (blue, green, orange, and red). Not sure what the
> difference is, but something definitely changed there.
Ah, I just noticed the bgColor field. I am pretty sure I am not doing anything with that. I'll fix that but :nanj says that doesn't affect him.
Comment 7•10 years ago
|
||
> Yeah, we removed that removed that feature because it isn't really used.
It is used for testing splice, so we (ops & qa) can redeploy existing tile distributions.
I personally use it to view the currently deployed tile set.
It was also extremely helpful in the case where a bad tile set was mistakenly published to prod, and we needed to roll back to the most recent correct deploy.
Is there any way I could get some sort of 'load most recent tile set' button? That would fix the first two use cases for me.
Regardless, this is non-blocking.
| Reporter | ||
Comment 8•10 years ago
|
||
(In reply to Daniel Thorn [:relud] from comment #7)
> Is there any way I could get some sort of 'load most recent tile set'
> button? That would fix the first two use cases for me.
We were discussing adding a way to look at previous distributions that would work just like the Upcoming view. But it sounds like you want to be able to publish those. So we should probably just add that feature back in. No big deal.
For seeing what's currently in prod, you can use this previewer: http://rlr.github.io/webtiles-previewer-redux/
Comment 9•10 years ago
|
||
previewer - nice! If I also had a link to download a standard tile set to deploy to stage, then I'd be set, no need to bring the previous distributions feature back.
Comment 10•10 years ago
|
||
marking this bug as resolved, because 1.1.35 is deployed in stage. Please file a new bug when the bgColor bug is fixed, and we can get this (awesome) UI update to prod.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•