Skip to content

Commit

Permalink
Brands removal
Browse files Browse the repository at this point in the history
  • Loading branch information
atanaskuzmanov-ft committed May 10, 2019
1 parent ed005b4 commit d14311e
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 15 deletions.
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,9 +58,6 @@ You should receive a response body like:
"authority": "http://api.ft.com/system/NEXT-VIDEO-EDITOR",
"identifierValue": "e2290d14-7e80-4db8-a715-949da4de9a07"
}],
"brands": [{
"id": "http://api.ft.com/things/dbb0bdae-1f0c-11e4-b0cb-b2227cce2b54"
}],
"mainImage": "ffc60243-2b77-439a-38af-98acd99af4ca",
"transcript": "<p>Here's what we're watching with trading underway in London. Global equities under pressure led by weaker commodities and financials as investors scrutinise the viability of the Trump trade. The dollar is weaker. Havens like yen, gold, and government bonds finding buyers. </p><p>As the dust settles over the failure to replace Obamacare, focus now on whether tax reform and other fiscal measures will eventuate. This is where the rubber meets the road for the Trump trade. High flying equity markets had been underpinned by the promise of big tax cuts and fiscal stimulus. And Wall Street is souring. </p><p>One big beneficiary of lower corporate taxes under Trump are small caps. They are now down 2 and 1/2% for the year. While the sector is still much higher since November, this is a key market barometer of prospects for the Trump trade. </p><p>Now while many still think some measure of tax reform or spending will eventuate, markets are very wary, namely of the risk that Congress and the Trump administration fail to reach agreement on legislation, that unlike health care reform, matters a great deal more to investors. </p><p>[MUSIC PLAYING] </p>",
"captions": [{
Expand Down
5 changes: 0 additions & 5 deletions video/model.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,13 @@ type identifier struct {
IdentifierValue string `json:"identifierValue"`
}

type brand struct {
ID string `json:"id"`
}

type videoPayload struct {
Id string `json:"uuid,omitempty"`
Title string `json:"title,omitempty"`
Standfirst string `json:"standfirst,omitempty"`
Description string `json:"description,omitempty"`
Byline string `json:"byline,omitempty"`
Identifiers []identifier `json:"identifiers,omitempty"`
Brands []brand `json:"brands,omitempty"`
FirstPublishedDate string `json:"firstPublishedDate,omitempty"`
PublishedDate string `json:"publishedDate,omitempty"`
MainImage string `json:"mainImage,omitempty"`
Expand Down
2 changes: 1 addition & 1 deletion video/test-resources/video-output.json
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"contentUri":"http://next-video-mapper.svc.ft.com/video/model/a40808ac-1417-4c48-9781-1dd2d8c8c6dc","payload":{"uuid":"a40808ac-1417-4c48-9781-1dd2d8c8c6dc","title":"ECB and Fed debates hit dollar and euro","standfirst":"Highlights of the key stories in the markets on Thursday","description":"The FT's Katie Martin highlights the main stories in the markets on Thursday, including the European Central Bank debating when to end its stimulus, and signs of pressure on the euro and the dollar.","byline":"Filmed by Nicola Stansfield. Produced by Vanessa Kortekaas.","identifiers":[{"authority":"http://api.ft.com/system/NEXT-VIDEO-EDITOR","identifierValue":"a40808ac-1417-4c48-9781-1dd2d8c8c6dc"}],"brands":[{"id":"http://api.ft.com/things/dbb0bdae-1f0c-11e4-b0cb-b2227cce2b54"}],"firstPublishedDate":"2017-04-06T09:58:35.440Z","publishedDate":"2017-04-12T12:29:48.331Z","mainImage":"77d86b7c-dfed-4a9b-3ad0-82f7908a56c5","transcript":"<p>From the FT in London, here's the latest on markets. The tussle over what the European Central Bank will do next continues. Yesterday, Germany's Jens Weidmann stuck faithfully to his national stereotypes, calling for the ECB to call time on its stimulus measures now that inflation has started to recover. </p><p>Enter stage right, ECB Chief Mario Draghi, who's clearly not convinced. Speaking today, he stressed that the rising inflation has been fragile, and says he sees no reason to tweak the Central Bank's usual script. The result of this swipe at the hawks-- well, the euro has dropped further $1.06 to the dollar. The debate is, of course, global. </p><p>Overnight minutes from the latest Fed meeting showed officials are pondering how to trim its $4.5 trillion balance sheet. That's been enough to deliver a jolt of nerves to US stocks. And Republicans are openly admitting now that tax reform will be hard. This is pressure for the dollar, with US currency making losses in particular against the yen. Watch oil hit again by record US stockpiles and the Trump-Xi meeting, which raises the possibility of barbs over currency policy. </p>","captions":[{"url":"https://next-video-editor.ft.com/783739.vtt","mediaType":"text/vtt"}],"dataSource":[{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/0x0.mp3","mediaType":"audio/mpeg","duration":68544,"audioCodec":"mp3"},{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/640x360.mp4","pixelWidth":640,"pixelHeight":360,"mediaType":"video/mp4","duration":68587,"videoCodec":"h264","audioCodec":"aac"},{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/1280x720.mp4","pixelWidth":1280,"pixelHeight":720,"mediaType":"video/mp4","duration":68587,"videoCodec":"h264","audioCodec":"aac"},{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/1920x1080.mp4","pixelWidth":1920,"pixelHeight":1080,"mediaType":"video/mp4","duration":68587,"videoCodec":"h264","audioCodec":"aac"}],"canBeDistributed":"yes","type":"Video","lastModified":"2017-04-13T10:27:32.353Z","publishReference":"tid_123123","canBeSyndicated":"yes","accessLevel":"free","webUrl":"https://www.ft.com/content/a40808ac-1417-4c48-9781-1dd2d8c8c6dc","canonicalWebUrl":"https://www.ft.com/content/a40808ac-1417-4c48-9781-1dd2d8c8c6dc"},"lastModified":"2017-04-13T10:27:32.353Z"}
{"contentUri":"http://next-video-mapper.svc.ft.com/video/model/a40808ac-1417-4c48-9781-1dd2d8c8c6dc","payload":{"uuid":"a40808ac-1417-4c48-9781-1dd2d8c8c6dc","title":"ECB and Fed debates hit dollar and euro","standfirst":"Highlights of the key stories in the markets on Thursday","description":"The FT's Katie Martin highlights the main stories in the markets on Thursday, including the European Central Bank debating when to end its stimulus, and signs of pressure on the euro and the dollar.","byline":"Filmed by Nicola Stansfield. Produced by Vanessa Kortekaas.","identifiers":[{"authority":"http://api.ft.com/system/NEXT-VIDEO-EDITOR","identifierValue":"a40808ac-1417-4c48-9781-1dd2d8c8c6dc"}],"firstPublishedDate":"2017-04-06T09:58:35.440Z","publishedDate":"2017-04-12T12:29:48.331Z","mainImage":"77d86b7c-dfed-4a9b-3ad0-82f7908a56c5","transcript":"<p>From the FT in London, here's the latest on markets. The tussle over what the European Central Bank will do next continues. Yesterday, Germany's Jens Weidmann stuck faithfully to his national stereotypes, calling for the ECB to call time on its stimulus measures now that inflation has started to recover. </p><p>Enter stage right, ECB Chief Mario Draghi, who's clearly not convinced. Speaking today, he stressed that the rising inflation has been fragile, and says he sees no reason to tweak the Central Bank's usual script. The result of this swipe at the hawks-- well, the euro has dropped further $1.06 to the dollar. The debate is, of course, global. </p><p>Overnight minutes from the latest Fed meeting showed officials are pondering how to trim its $4.5 trillion balance sheet. That's been enough to deliver a jolt of nerves to US stocks. And Republicans are openly admitting now that tax reform will be hard. This is pressure for the dollar, with US currency making losses in particular against the yen. Watch oil hit again by record US stockpiles and the Trump-Xi meeting, which raises the possibility of barbs over currency policy. </p>","captions":[{"url":"https://next-video-editor.ft.com/783739.vtt","mediaType":"text/vtt"}],"dataSource":[{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/0x0.mp3","mediaType":"audio/mpeg","duration":68544,"audioCodec":"mp3"},{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/640x360.mp4","pixelWidth":640,"pixelHeight":360,"mediaType":"video/mp4","duration":68587,"videoCodec":"h264","audioCodec":"aac"},{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/1280x720.mp4","pixelWidth":1280,"pixelHeight":720,"mediaType":"video/mp4","duration":68587,"videoCodec":"h264","audioCodec":"aac"},{"binaryUrl":"http://ftvideo.prod.zencoder.outputs.s3.amazonaws.com/783749/1920x1080.mp4","pixelWidth":1920,"pixelHeight":1080,"mediaType":"video/mp4","duration":68587,"videoCodec":"h264","audioCodec":"aac"}],"canBeDistributed":"yes","type":"Video","lastModified":"2017-04-13T10:27:32.353Z","publishReference":"tid_123123","canBeSyndicated":"yes","accessLevel":"free","webUrl":"https://www.ft.com/content/a40808ac-1417-4c48-9781-1dd2d8c8c6dc","canonicalWebUrl":"https://www.ft.com/content/a40808ac-1417-4c48-9781-1dd2d8c8c6dc"},"lastModified":"2017-04-13T10:27:32.353Z"}
6 changes: 0 additions & 6 deletions video/video_mapper.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ const (
videoType = "Video"
videoContentURIBase = "http://next-video-mapper.svc.ft.com/video/model/"
videoAuthority = "http://api.ft.com/system/NEXT-VIDEO-EDITOR"
ftBrandID = "http://api.ft.com/things/dbb0bdae-1f0c-11e4-b0cb-b2227cce2b54"
dateFormat = "2006-01-02T15:04:05.000Z0700"
defaultAccessLevel = "free"
uuidGenerationSalt = "storypackage"
Expand Down Expand Up @@ -108,10 +107,6 @@ func getVideoModel(videoContent map[string]interface{}, uuid string, tid string,
IdentifierValue: uuid,
}

b := brand{
ID: ftBrandID,
}

accessLevel := getAccessLevel()

webUrl := fmt.Sprintf(webUrlTemplate, uuid)
Expand All @@ -124,7 +119,6 @@ func getVideoModel(videoContent map[string]interface{}, uuid string, tid string,
Description: description,
Byline: byline,
Identifiers: []identifier{i},
Brands: []brand{b},
FirstPublishedDate: firstPublishDate,
PublishedDate: publishedDate,
MainImage: mainImage,
Expand Down

0 comments on commit d14311e

Please sign in to comment.