1. Good question Jess. We could solve that by having a special token like "none". 2. We can't use `context` for this purpose, because that is currently being used by Firefox clients today and need to keep it around for a while to support backwards compatibility. We could call the new field `sponsored_by_override`.
Bug 1618064 Comment 9 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
1. Good question Jess. We could solve that by having a special token like "none" that would return an empty string. Not setting the value in AdZerk would result in dropping the field altogether. 2. We can't use `context` for this purpose, because that is currently being used by Firefox clients today and need to keep it around for a while to support backwards compatibility. We could call the new field `sponsor_by_override`. Proposed logic: - If `sponsor_by_override` has a value then use it as-is (eg: if `sponsor_by_override` is "Brought to you by Mozilla", then just show that) - If `sponsor_by_override` is an empty string then don't show the label. - If `sponsor_by_override` is missing, then we will show "Sponsored by " + sponsor in the client.
1. Good question Jess. We could solve that by having a special token like "none" that would return an empty string. Not setting the value in AdZerk would result in dropping the field altogether. 2. We can't use `context` for this purpose, because that is currently being used by Firefox clients today and need to keep it around for a while to support backwards compatibility. We could call the new field `sponsored_by_override`. Proposed logic: - If `sponsored_by_override` has a value then use it as-is (eg: if `sponsored_by_override` is "Brought to you by Mozilla", then just show that) - If `sponsored_by_override` is an empty string then don't show the label. - If `sponsored_by_override` is missing, then we will show "Sponsored by " + sponsor in the client.
1. Good question Jess. We could solve that by having a special token like "none" that would return an empty string. Not setting the value in AdZerk would result in dropping the field altogether. 2. We can't use `context` for this purpose, because that is currently being used by Firefox clients and we need to keep it around for a while to support backwards compatibility. We could call the new field `sponsored_by_override`. Proposed logic: - If `sponsored_by_override` has a value then use it as-is (eg: if `sponsored_by_override` is "Brought to you by Mozilla", then just show that) - If `sponsored_by_override` is an empty string then don't show the label. - If `sponsored_by_override` is missing, then we will show "Sponsored by " + sponsor in the client.