Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hosted Native Traffic Driver #160

Open
JonNorman opened this issue Oct 4, 2017 · 3 comments
Open

Hosted Native Traffic Driver #160

JonNorman opened this issue Oct 4, 2017 · 3 comments

Comments

@JonNorman
Copy link

There are two different formats in DFP:

But we only have a single format in this repository, which appears to be a superset of all the fields in each format (there are some fields that are only needed for AMP or Web).

The format in DFP should be altered to reflect the test.json contents. Some of the fields that match between both formats in DFP are spelled slightly differently and so the templates themselves may need to be updated e.g. Brand Colour vs Brand Colour, Article Headline vs Title.

@JonNorman
Copy link
Author

@RobertFreeman do you know whether this format (and associated styles) are even used? If not then it may be worth just deleting them.

@RobertFreeman
Copy link
Collaborator

It's up to you Jon - I'm not planning on optimising any Labs promotion on AMP any time soon, that's not to say we definitely won't in the future though. I am super into this clean up project though, so if you're on a mission to have single formats with multiple styles then might be a good one to do. But def don't prioritise it over anything important!

@JonNorman
Copy link
Author

Great - thanks Rob. I might just do it as part of my style-sync work. I want to release this tool when everything is nice and consistent and ordered such that we can use any discrepancies as warnings to stop pushing up changes etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants