Allow manual configuration

  • Resolved Corey Worrell

    (@coreyw)


    5 months, 3 weeks ago

    It would be great to allow an option/setting to input other configuration values for the tracking script. We often need to include Google Ads remarketing tag: gtag('config', 'AW-1645116xxxx'); and currently have to use a filter in code. But adding through dashboard would be much more flexible. I understand we could use Google Tag Manager or similar, but since GADWP is already handling the script, it feels better to keep it there.

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Alin Marcu

    (@deconf)

    5 months, 2 weeks ago

    Hi and thanks for the suggestion.

    For security reasons I try to avoid as much as possible outputting a user input with low sanitizing/escaping which is a must for such a feature. In exchange we could add an option for this particular requirement. Would that help?

    Thread Starter Corey Worrell

    (@coreyw)

    5 months, 2 weeks ago

    I understand. Do you mean you would add a database option but not provide a UI for it? That would effectively be the same as it is now, since there is filter to hook into, I could add my own UI (ACF for example) to save the value and then feed it into that filter hook.

    Plugin Author Alin Marcu

    (@deconf)

    5 months, 2 weeks ago

    Nope, an UI option and an input field for the AW-XXXXXX value. Once enabled it will output the needed code. This can be properly sanitized and escaped.

    What isn’t safe and hard to make it safe enough is allowing the user to enter a code snippet which would be output by the plugin within the tracking code.

    Thread Starter Corey Worrell

    (@coreyw)

    5 months, 2 weeks ago

    Oh I see what you mean. That could be valuable, but there are other config options that Google allows you to set as well. So I could see it being unwieldy to add a field for each. For now I can just disable the tracking code and add it manually via Tag Manager or custom snippet plugin. Thank you!

Viewing 4 replies - 1 through 4 (of 4 total)
  • You must be logged in to reply to this topic.