Project

General

Profile

Bug #3573

Kampagne PLUS : Muliggørelse af kampagne-tracking

Added by Tue Gaston over 1 year ago. Updated 3 months ago.

Status:
Resolved (tag version)
Priority:
High
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Kampagner

Description

I fb m udviklingen af modulet Kampagne PLUS https://platform.dandigbib.org/projects/kampagne-plus ønskes der mulighed for tracking af kampagner - ikke kun hvor ofte der klikkes på dem, men også hvor ofte de vises.

Dette kræver at Kampagne PLUS sættes op, så der kan indsættes af en page parameter i hver kampagne.

Screen Shot 2018-11-22 at 12.55.22.png (177 KB) Screen Shot 2018-11-22 at 12.55.22.png Jesper Kristensen, 11/22/2018 12:56 PM
webtrekk-tiloader-htmlhead.PNG (128 KB) webtrekk-tiloader-htmlhead.PNG Simon Holt, 11/30/2018 03:15 PM
3573-campaing-tracking-tag.PNG (27.8 KB) 3573-campaing-tracking-tag.PNG Simon Holt, 07/17/2019 12:36 PM
3573-event-send-only-once.PNG (20.3 KB) 3573-event-send-only-once.PNG Simon Holt, 09/02/2019 03:10 PM

Related issues

Related to DDB CMS - Enhancement #2177: Kampagne PLUSResolved (tag version)
Related to DDB CMS - Enhancement #4443: Ændring af beskrivelse i Kampagne PLUSClosed
Related to DDB CMS - Enhancement #4445: Kampagne PLUS: Fjernelse af feltet "Tracking tags"Resolved (tag version)
Has duplicate DDB CMS - Bug #3369: Kampagne PLUS tilpasning til webbtrekClosed

History

#1 Updated by Tue Gaston over 1 year ago

  • Description updated (diff)

#2 Updated by Ture Gjørup over 1 year ago

I forhold til tracking af visning af kampanger har vi haft en dialog med Tue/Piotr om hvordan det kunne implementeres.

Piotr har tidligere overfor Tue skitseret to muligheder at tracke visninger på, enten med WebTrekk page parameter/variabel nr. 11 eller med Teaser-Tracking-Plugin, og umiddelbart er konklussionen at vi skal forsøge os med i ”variabel nr. 11” løsningen.
 
Jeg har kigget på: https://support.webtrekk.com/hc/en-us/articles/115004108325-How-to-set-up-custom-parameters og https://support.webtrekk.com/hc/en-us/article_attachments/115007805409/Webtrekk-Implementation-Guidelines-v6-EN.PDF - 3.3 Custom page parameters. Men kan ikke  umiddelbart se hvad der skal til i DDB for at sætte ”variabel nr. 11”.

Jeg antager at det er dette der skal udvides:

<!--
window._ti = window._ti || {}
//-->

Men vil høre om Piotr har et par pointers til hvordan WebTrekk konfigurationen i DDB skal sættes?

 

#3 Updated by Michael Ødum over 1 year ago

Først kampagnen, lige nu har vi en kildekode med OnSiteSearch som ser således ud på f.eks. søgeresultatsiderne:

 

<script type="text/javascript">

<!--//--><![CDATA[//><!--

window._ti = window._ti || {};

window._ti['pageTitle'] = 'Search ting’;

window._ti['oss'] = ‘harry potter'

//--><!]]>

</script>

 

Den skal ganske rigtigt udvides, så den får tilføjet en ekstra window._ti parameter på:

 

<script type="text/javascript">

<!--//--><![CDATA[//><!--

window._ti = window._ti || {};

window._ti['pageTitle'] = 'Search ting’;

window._ti['oss'] = ‘harry potter'

window._ti['p_campaign_show'] = ‘Min_kampagne'

//--><!]]>

</script>

 

”Min_kampagne” er her navnet på kampagnen.

I behøver ikke de 2 forgående linjer for at det virker, men vil opfordre til vi får det testet sammen

#5 Updated by Ture Gjørup over 1 year ago

Tak for update Michael. Vi har nu haft tid til atkigge på opgaven og kan se at vi rammer et problem i forhold til rækkefølgen af load af siden.  Kampanger lazyloades, hvilket praktisk betyder at på det tidspunkt hvor webtrekk modulet kører er det endnu om eller hvilken kampange der skal vises på pågældende side. På det tidspunkt hvor kampangen loades er det for sent at indsætte javascript på siden.

Derfor har vi lavet en løsning hvor vi bruger sender kampange koden som et javascript event efter siden er loadet i browseren. Vi har fulgt Webtrekk-Implementation-Guidelines-SPA-EN.pdf, og taget udgnagspunkt i "4.4 Campaign information". Dvs vi bruger "wt.sendinfo" til at sende "campaignId" og "campaignAction":

/**
 * @file
 * Added webtrekk view tracking to campaigns.
 */
(function ($) {
  'use strict';

  // Use custom event fired when campaign is loaded to track the campaign.
  $(document).on('campaignPlusLoaded', function (event, campaignId) {
    // The global wt (webtrekk) object is only loaded on approved domains. So to
    // not block execution test for the variable.
    if (typeof(wt) !== 'undefined') {
      wt.sendinfo({
        campaignId: campaignId,
        campaignAction: 'view'
      });
    }
  });
})(jQuery);

Spørgsmålet er så om dette vil give de ønskede tracking og statistisk muligheder i Webtrekk?

 

#6 Updated by Tue Gaston over 1 year ago

  • Assignee set to Piotr Birketoft

#8 Updated by Piotr Birketoft about 1 year ago

Jeg har været inde og undersøge siden https://kampagne.srvitkphp56stg.itkdev.dk/search/ting/harry%20potter

Jeg kan ikke se webtrekk tracking koden på siden hvis jeg kigger i developer tools.

Ydermere kan jeg ikke få nogen resultater hvis jeg søger på "campaignId" eller " window._ti "

 

Kigger jeg det rigtige sted?

 

#9 Updated by Anna Christensen about 1 year ago

Koden er i mellemtiden blevet merget til core og lagt op til test på: https://upgrade-fbs.ddbcms.dk/

Der er allerede oprettet og udløst en række kampagner på dette site som burde generere noget statistik. Der udløses fx. kampagner v. søgning på: Genbrug, Krimi, Fantasy og Historie.

Håber det giver pote ellers må du lige vende tilbage.

#10 Updated by Piotr Birketoft about 1 year ago

Hej Anna,

Tak for hurtigt svar retur. Jeg har kigget igen, og vi kan desværre stadig ikke se scriptet. Kan du verificere at det er implementeret?

PS: Kan du evt henvise til hvilken linje i koden scriptet ligger på, så kan jeg være med til at checke om det er korrekt formateret.

- Piotr

#11 Updated by Piotr Birketoft about 1 year ago

Hej Anna,

 

Jeg har lagt et kodeeksempel op på, hvordan at kampagnevisning kan sættes op.

Du kan se det her:

http://www.kpiindex.com/oss7_campaign.html

 

De er testet og samler data op.

#12 Updated by Jesper Kristensen about 1 year ago

Som jeg ser det er webtrekk ikke slået til på ddb tests sites?

@rolf @gitte hvordan er det med trakcing og dev sites?

#13 Updated by Jesper Kristensen about 1 year ago

  • Assignee changed from Piotr Birketoft to Gitte Barlach

#14 Updated by Tue Gaston about 1 year ago

Jesper Kristensen wrote:

Som jeg ser det er webtrekk ikke slået til på ddb tests sites?

@rolf @gitte hvordan er det med trakcing og dev sites?

Jeg har skyndt mig at slå den til.
Under konfigurationen mangler jeg TagIntegration Domain og TagIntegration ID:

Jeg ringer til Piotr med det samme.

#15 Updated by Tue Gaston about 1 year ago

  • Assignee changed from Gitte Barlach to Piotr Birketoft
  • Priority changed from Normal to High

#16 Updated by Piotr Birketoft about 1 year ago


Hej med jer,

 

Hermed tag integration domain:

ddbtest01.wt-eu02.net

 

Track Id:

267582424135824

 

 

#17 Updated by Tue Gaston about 1 year ago

Piotr Birketoft wrote:


Hej med jer,

 

Hermed tag integration domain:

ddbtest01.wt-eu02.net

 

Track Id:

267582424135824

 

 

Så har jeg sat den op, og nu skete der noget med https://upgrade-fbs.ddbcms.dk/search/ting/fantasy?.

#18 Updated by Tue Gaston about 1 year ago

  • Assignee changed from Piotr Birketoft to Jesper Kristensen

#19 Updated by Piotr Birketoft about 1 year ago

Tue og jeg har nu rekonfigureret trackingen, så den sender grunddata afsted, og vi derved kan teste om kampagnen er implementeret.

Det er den desværre ikke.

 

Må jeg foreslå at i med skælnen til http://www.kpiindex.com/oss7_campaign.html implementerer:



	window._ti['p_campaign_show'] = 'Mit Kampagne navn';	

Et sted i mellem linje 38 -til linje 42 på view-source:https://upgrade-fbs.ddbcms.dk/search/ting/fantasy 


<script type="text/javascript">
<!--//--><![CDATA[//><!--
window._ti = window._ti || {};
window._ti['pageTitle'] = 'Search ting';
window._ti['p_campaign_show'] = 'Mit Kampagne navn';  <--- tilføjet linje
window._ti['OSS'] = 'fantasy';
window._ti['OSSr'] = '5580'
//--><!]]>
</script>

Sig til hvis i har brug for et kald og en snarlig test!

 

Hilsen,

Piotr

#20 Updated by Tue Gaston about 1 year ago

  • Assignee changed from Jesper Kristensen to Ture Gjørup

#21 Updated by Jesper Kristensen about 1 year ago

  • Assignee changed from Ture Gjørup to Gitte Barlach

#22 Updated by Gitte Barlach about 1 year ago

  • Parent task deleted (#3369)

#23 Updated by Gitte Barlach about 1 year ago

  • Tracker changed from Support to Bug
  • Project changed from Webtrekk to DDB CMS
  • Status changed from New to Needs code review
  • Assignee changed from Gitte Barlach to Kasper Garnæs
  • Target version set to Release 30 - BPI, Kampagneplus og Sektioner - (ITK)
  • Parent task set to #3369
  • Start date deleted (05/24/2018)
  • Kategorier Inspiration - Kampagner added

#24 Updated by Tue Gaston about 1 year ago

#25 Updated by Kasper Garnæs about 1 year ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Jesper Kristensen

Den foreslåede ændring i PR'et addresserer et problem hvor vi forsøger at tracke kampagner for Webtrekk er loaded.

Dette problem kan være forårsaget af at Webtrekk ikke er blevet integreret i overensstemmelse med implementation guidelines og derfor først loades for sent. Ud fra #3076 og tilhørende PR kan jeg ikke læse mig frem til hvorfor det skulle være tilfældet.

Hvis der en god grund til det bør den foreslåede løsning hæves til et niveau hvor den kan bruges af alle moduler der skal integreres med Webtrekk. Alternativt bør vores integration opdateres så den i højere grad føler guidelines.

#26 Updated by Gitte Barlach about 1 year ago

Se #3901, samt Simon´s kommentar 8 i #3632 ift.  "Vil mene det er bedre at håndtere tracking i ding_webtrekk modulet, som også er udgangspunkt i #3901. I hvert fald moduler der er med i core, bør tracking kode placeres i ding_webtrekk."

#27 Updated by Simon Holt about 1 year ago

Har ifb. med #3901 har jeg også opdateret ding_webtrekk så webtrekk-loaderen placeres i toppen af HTML-head lige under page parameters, som det er beskrevet og vist i vejledningen Kasper linker til. (se vedhæfning og commit: https://github.com/vejlebib/ding2/commit/08030b7b5b970dbfded55808be1feb428c701472)

Så muligvis vil det komme problemet her til livs, da scriptet nu skulle loades hurtigere. 

#28 Updated by Gitte Barlach about 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

Kan dette issue godkendes på baggrund af den løsning Simon har lavet?

#29 Updated by Kasper Garnæs about 1 year ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Jeg vil mene at ændringen i PR'et i dette issue bør være unødvendigt hvis løsningen fra #3901 godkendes og merges.

Mit forslag er derfor at vi sætte dette issue til test samtidig med #3901.

#30 Updated by Simon Holt about 1 year ago

Lige en lille bemærkning: Kan se der anvendes den gamle metode til at sende events med wt.sendinfo(). 

I forbindelse med arbejdet i #3901 fandt vi ud af det ikke spillede så godt på den måde, og metoden og strukturen af det data, der skal sendes med, er ændret. Se eksempel her: https://github.com/ding2/ding2/pull/1283/files#diff-1b9f9b1b50718dcbd760ca804d2d6016R41

Men jeg ville nok lige vende den med Piotr igen og sikrer i sender på den rigtige måde.

#31 Updated by Simon Holt about 1 year ago

Ah, har vist misforstået hvad planen var her.

Jeg vil foreslå at PR her opdateres, så det med at prøve at fikse timingen fjernes og at I bare sender event afsted når kampagnen er loadet. Med rettelserne i #3901 bør Webtrekk loade betydeligt hurtigere og jeg tror altså ikke der vil være nogle problemer.

Men som sagt skal i nok lige i dialog med Piotr. For det hjælper ikke bare at sende noget afsted. Det skal aftales helt præcist hvad for en parameter i vil bruge og hvad for nogle værdier der skal sendes. Ellers bliver det aldrig sendt til Webtrekk og registreret. Som sagt er vi i #3901 gået til at bruge push() metoden i stedet for sendinfo() metoden. Kan godt være I også skal gøre det samme her, men hør Piotr ad.

#32 Updated by Gitte Barlach about 1 year ago

  • Assignee changed from Gitte Barlach to Tue Gaston

Tue kan du tage kontakt til Piotr ift. hvilken metode, der er den korrekte? 

#33 Updated by Piotr Birketoft about 1 year ago

Hej med jer,

Jeg har nu undrersøgt siden https://kampagne.srvitkphp56stg.itkdev.dk/search/ting/harry%20potter

og jeg kan ikke se at der bliver afviklet nogen tracking fra den side.

Hvis jeg søger på "Campaign" får jeg hellere ikke nogen søgeresultater frem.

Hilsen,

Piotr

 

 

#34 Updated by Tue Gaston about 1 year ago

Piotr Birketoft wrote:

Hej med jer,

Jeg har nu undrersøgt siden https://kampagne.srvitkphp56stg.itkdev.dk/search/ting/harry%20potter

og jeg kan ikke se at der bliver afviklet nogen tracking fra den side.

Hvis jeg søger på "Campaign" får jeg hellere ikke nogen søgeresultater frem.

Hilsen,

Piotr

 

 

Den kampagne der blev udløst af søgning på "harry potter" på https://kampagne.srvitkphp56stg.itkdev.dk/ var åbenbart blevet slettet eller ændret.
Jeg har nu rettet en af de gamle kampagner til, så den udløser på "harry potter".

#35 Updated by Tue Gaston about 1 year ago

  • Assignee changed from Tue Gaston to Piotr Birketoft

#36 Updated by Tue Gaston about 1 year ago

Efter samtale med Simon og Rolf og Christel er det nu besluttet at skyde kampagne tracking til release 31.

#37 Updated by Tue Gaston about 1 year ago

  • Subject changed from Muliggørelse af kampagne-tracking vha. page parameters to Muliggørelse af kampagne-tracking
  • Status changed from Reviewed - Needs info/rework to Ready for development
  • Assignee changed from Piotr Birketoft to Simon Holt
  • Priority changed from High to None
  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner - (ITK) to Release 31 - bugfixes

#38 Updated by Tue Gaston about 1 year ago

  • Subject changed from Muliggørelse af kampagne-tracking to Kampagne PLUS : Muliggørelse af kampagne-tracking

#39 Updated by Tue Gaston 8 months ago

  • Priority changed from None to High

#40 Updated by Tue Gaston 6 months ago

  • Parent task deleted (#3369)

#41 Updated by Tue Gaston 6 months ago

  • Has duplicate Bug #3369: Kampagne PLUS tilpasning til webbtrek added

#42 Updated by Piotr Birketoft 6 months ago

Kære alle,

 

Både https://kampagne.srvitkphp56stg.itkdev.dk/search/ting/harry%20potter  og https://kampagne.srvitkphp56stg.itkdev.dk/install.php?profile=ding2&locale=en

melder fejl på sitet.

 

Ring/Skriv gerne, så tester og supporterer jeg det jeg kan :)
 

#43 Updated by Tue Gaston 6 months ago

Piotr Birketoft wrote:

Kære alle,

 

Både https://kampagne.srvitkphp56stg.itkdev.dk/search/ting/harry%20potter  og https://kampagne.srvitkphp56stg.itkdev.dk/install.php?profile=ding2&locale=en

melder fejl på sitet.

 

Ring/Skriv gerne, så tester og supporterer jeg det jeg kan :)
 

Nej, det er der ikke noget mærkeligt i.
Kampagne PLUS er jo nu rullet ud, så det dedikerede test-site er lukket.

Jeg har lige oprettet et par kampagner på vores almindelige test-site, som du kan fornøje ansigtet med:

https://upgrade-fbs.ddbcms.dk/search/ting/term.type%3D%22film%20%28net%29%22?
https://upgrade-fbs.ddbcms.dk/search/ting/term.acsource%3D%22science%2A%22?

Har du dermed, hvad du skal bruge?

#44 Updated by Piotr Birketoft 5 months ago


Hej Tue,

Desværre ikke.

De 2 sider ud har sendt link til giver begge ingen kampagne " Din søgning desværre ikke gav noget resultat, men vi vil gerne hjælpe dig med at komme videre! "

 

Parameterne - og jeg har sat kampagne plus til parameter 45, den ser jeg hellere ikke:

p 505,upgrade-fbs_ddbcms_dk.search.ting.term_acsource%3d%22science%2a%22,1,1920x1080,24,1,1562235514942,https://platform.dandigbib.org/issues/3573,1712x360,0
tz 2
eid 2154296400267705299
one 0
fns 0
geid 815622345260092375623756
la en
is term.acsource="science*"
cp10 term.acsource="science*"
cp11 0
cp21 1
cp39 0
cp771 0
pu https://upgrade-fbs.ddbcms.dk/search/ting/term.acsource%3D%22science%2A%22
eor 1

Misser jeg noget?

 

#45 Updated by Tue Gaston 5 months ago

Nej, det er der ikke noget at sige til - vores test-site er blevet bygget på ny, siden jeg oprettede kampagnerne.

Der er derfor ingen kampagner - og der er også nogen, der har pillet ved nogle indstillinger, så der lige pt slet ikke kan fremsøges noget som helst (vi sidder jo nogen stykker og tester den nyeste release).

Jeg undersøger det, opretter en ny kampagne, og vender tilbage.

#46 Updated by Piotr Birketoft 5 months ago


Tak Tue.

#47 Updated by Tue Gaston 5 months ago

Piotr Birketoft wrote:


Tak Tue.

Hov vent, du kan jo bare bruge vores wm-site i stedet.

Her er en kampagne: https://wm.ddbcms.dk/search/ting/term.subject%3Dromerriget?

Du skal ikke tage dig af boksen med forslag fra bibliotek.dk

#48 Updated by Simon Holt 5 months ago

Foreløbigt PR: https://github.com/ding2/ding2/pull/1480

Tilføjer også ny debug mode der outputter event data i Javascript konsollen til hjælp når vi skal udvikle.

Sætter ikke til code review endnu. Piotr skal lige kigge på det og så er der et par spørgsmål som jeg lige tager med ham.

Test tracking af kampagner her: https://ddbcms.vejlebib.dk/search/ting/lydb%C3%B8ger (er hooket op med Webtrekk så events skulle blive sendt).

#49 Updated by Simon Holt 5 months ago

PR er testet af Piotr og han bekræfter der kommer korrekt data om viste kampagner ind fra: https://ddbcms.vejlebib.dk/search/ting/lydb%C3%B8ger

Mangler lige lidt input fra forretningen, før jeg sætter PR til code review:

Skal det være muligt at angive flere tracking tags pr. kampagne?

I så fald vil jeg foreslå vi separerer dem med komma og opdaterer beskrivelsen af feltet, så det er tydeligt det understøtter flere tags og at man skal bruge komma.

Hvis ikke vi vil understøtter flere tags, skal beskrivelsen også opdateres så det ikke er flertal og man dermed ledes til at tro, at der understøttes flere tags.

#50 Updated by Simon Holt 5 months ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Simon Holt to Tue Gaston

#51 Updated by Tue Gaston 5 months ago

Simon Holt wrote:

PR er testet af Piotr og han bekræfter der kommer korrekt data om viste kampagner ind fra: https://ddbcms.vejlebib.dk/search/ting/lydb%C3%B8ger

Mangler lige lidt input fra forretningen, før jeg sætter PR til code review:

Skal det være muligt at angive flere tracking tags pr. kampagne?

I så fald vil jeg foreslå vi separerer dem med komma og opdaterer beskrivelsen af feltet, så det er tydeligt det understøtter flere tags og at man skal bruge komma.

Hvis ikke vi vil understøtter flere tags, skal beskrivelsen også opdateres så det ikke er flertal og man dermed ledes til at tro, at der understøttes flere tags.

 

Jamen du har jo som sædvanlig så evigt ret, kære Simon.

Min holdning er den, at eftersom jeg ikke kan komme på et scenarie, hvor man har brug for mere end et tag på en kampagne, så vi springer over hvor gærdet er lavest, og ændrer beskrivelsen fra "Tracking tags" til "Tracking tag".

Jeg har oprettet et issue på denne ganske lille rettelse: https://platform.dandigbib.org/issues/4443

#52 Updated by Tue Gaston 5 months ago

#53 Updated by Simon Holt 5 months ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Tue Gaston to Gitte Barlach

Ok, tak for det. Tracking tags kunne bruges på samme måde som "almindelige" tags, men giver dig ret. Et tag burde være tilstrækkeligt. Spørgsmålet er så, om det skal være påkrævet, men lad os tage den i den anden sag.

Har lige opdateret PR og da Piotr samtidig har verificeret at det virker på https://ddbcms.vejlebib.dk/search/ting/lydb%C3%B8ger, burde det hele være klar så sætter den til code review.

#54 Updated by Simon Holt 5 months ago

Efter samtale med Piotr og Tue, blev det aftalt at vi vil sende følgende data-layer object til webtrekk ved tracking af kampagner:

{
  "linkId": "Kampagne",
  "customClickParameter": {
    "62": [Kampagnens titel],
    "63": [Kampagnens interne ID],
    "64": [Kampagnens tracking tag]
  }
}

Som jeg har fremhævet, sender vi kampagnens tracking tag (se også vedhæfning) til Webtrekk, når den bliver vist på en side. Dette er sikkert også fint nok, men det ser egentlig ud til at tracking tag oprindeligt var tiltænkt som URL-parameter kampagnens "destination/link". F.eks. hvis vi har en kampagne der linker til en nyhed med information: nyheder/boeger/nye-nordiske-romaner, så vil det link der bliver genereret se ud som følgende (eksempel):

http://ddbcms.test/nyheder/boeger/nye-nordiske-romaner?wt_mc=[Kampagnens tracking tag]

Det er selvfølgelig rigtig smart, så vi kan se hvilke interne sidevisninger, der er udløst af en kampagne og kan vel fint fungere ved siden af, at vi også sender tag når vi sender kampagne event.

Der er dog nogle fejl i den måde det er implementeret på:

1. Den bruger parameteren wt_mc, som vist ikke er sat op i Webtrekk og derfor ikke fungerer korrekt. En anden parameter vi kunne bruge i stedet, som er sat op, er u_navigatedby, som vi anvender ved karuseller så vi kan se hvilke karuseller materialevisninger stammer fra. Tænker den vil give god mening her også? @Piotr har du noget input til dette? Vil det give mening at anvende u_navigatedby her?

2. URL-parameteren tilføjes ikke korrekt. Den appendes bare altid til URL med "?", hvilket betyder, at hvis man har kampagne link der i forvejen har en URL-parameter virker det ikke, da separatoren i dette tilfælde så skulle have været "&". Så hvis man f.eks. har en kampagne der link til en søgning med en søgeprofil vil det ikke virke da link vil blive: 

http://ddbcms.test/search/ting/?profile=bøger?wt_mc=[tracking tag]

og det skulle have været

http://ddbcms.test/search/ting/?profile=bøger&wt_mc=[tracking tag]

Punkt 2. er dermed en mindre alvorlig fejl, som gør at alle kampagne der har links med URL-parameter ikke vil virke.

Jeg foreslår vi får styr på denne URL-parameter i denne sag?

 

#55 Updated by Simon Holt 5 months ago

  • Status changed from Needs code review to Need more info
  • Assignee changed from Gitte Barlach to Tue Gaston

#56 Updated by Tue Gaston 5 months ago

  • Related to Enhancement #4445: Kampagne PLUS: Fjernelse af feltet "Tracking tags" added

#57 Updated by Simon Holt 5 months ago

  • Status changed from Need more info to Needs code review

PR er opdateret og klar til review: https://github.com/ding2/ding2/pull/1480 

Vi har løst det med URL-parameter ved at bruge u_navigatedby istedet for wt_mc. URL-parameteren tilføjes nu også korrekt ved at tage højde for eksisterende parametre.

Vi har valgt helt at skrotte tracking tag feltet og det er bare kampagnens titel der sendes med i u_navigatedby. Dermed sendes tag heller ikke når en kampagne vises og vi sender kun titel og intern ID:

{
  "linkId": "Kampagne",
  "customClickParameter": {
    "62": [Kampagnens titel],
    "63": [Kampagnens interne ID]
  }
}

Fjernelse af tracking tag feltet sker i: #4445

#58 Updated by Tue Gaston 4 months ago

  • Assignee changed from Tue Gaston to Gitte Barlach

#59 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#60 Updated by Piotr Birketoft 4 months ago


Hej Simon og Tue,

 

Jeg er enig i at bruge u_navigatedby= [Kampagnens tracking tag] i stedet for wt_mc= [Kampagnens tracking tag]

Er det et sted jeg kan teste den, for vi skal stadig lige sikre at det, at en kampagne bliver vist, fortsat trackes?

- Piotr

 

#61 Updated by Simon Holt 4 months ago

Hej Piotr,

Kampagnervisninger bliver fortsat tracket. Vi sender bare ikke tracking tag, men kun title og intern ID (se #note-57).

Du kan teste her: https://ddbcms.vejlebib.dk/search/ting/lydbøger 

 

#62 Updated by Piotr Birketoft 4 months ago

Tak Simon,

Jeg ser følgende:

KampagnePlus ID (Event Parameer 63): 58

KampagnePlus Title (Event Parameer 62): Lydbøger

Jeg kan også se at vi har fået rettet til, så URL linket får parameteren Navigated by:

https://ddbcms.vejlebib.dk/search/ting/term.subject%3Dkrimi?profile=lydboeger&u_navigatedby=Lydbøger

 

Potentielt problem:

"u_navigatedby" anvendes også i karouselen. Så vi får svært ved at skille Karousel clicks fra KampagnePlus clicks.

Hvis det er et issue, må jeg forslå at parameteren ændres til "u_navigatedby_kp"?

#63 Updated by Simon Holt 4 months ago

Hvis det ikke er noget problem at indstille en ny URL-parameter i Webtrekk, synes jeg helt klart vi skal bruge en specifik u_navigatedby_kp til kampagner.

#64 Updated by Simon Holt 4 months ago

@Piotr er det noget du vil sætte op? For så vil jeg opdatere PR med den nye URL-parameter inden Kasper skal kigge på koden.

#65 Updated by Kasper Garnæs 4 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Simon Holt

Jeg har set lidt på koden nu og jeg har et par kommentarer, som måske også kan nå at komme med.

#66 Updated by Tue Gaston 4 months ago

Kasper Garnæs wrote:

Jeg har set lidt på koden nu og jeg har et par kommentarer, som måske også kan nå at komme med.

Jeg kan se på github at @simon har svaret på alle kommentarerne, og at @kasper har givet alle Simons svar thumbs up.
Betyder det, at dette issue skal have status "Needs Code Review" igen eller hvad?

#67 Updated by Simon Holt 4 months ago

Hej Tue, gi mig en lige times tid, så er det klar. Der var lige nogle ting der drillede.

#68 Updated by Simon Holt 4 months ago

Så er PR klar til review igen: https://github.com/ding2/ding2/pull/1480

Der var som sagt nogle ting der drillede og så havde jeg også nogle problemer med nogle campaign_plus behat tests.. men lige pludselig kørte testen igennem uden problemer og ved fandt aldrig ud af hvad problemet var. Måske nogle problemer med circle eller noget..

#69 Updated by Simon Holt 4 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

Der er stadig et enkelt Scrutinizer issue, som jeg vil foreslå vi ignorere. Mere info her: https://github.com/ding2/ding2/pull/1480#issuecomment-521687648

#70 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#71 Updated by Kasper Garnæs 4 months ago

  • Status changed from Needs code review to Technical test

Reviewed og godkendt.

#72 Updated by Kasper Garnæs 4 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#73 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Gitte Barlach to Tue Gaston

#74 Updated by Tue Gaston 3 months ago

Jeg er i tvivl om, hvordan jeg tester denne.
Når jeg udløser en kampagne-PLUS-kampagne ved navn "Kampagne plus" på https://upgrade-fbs.ddbcms.dk/ (der er på vers. ding2-7.x-4.8.0-rc5), så kan jeg se, at flg. sendes til Webtrekk:

GET /267582424135824/wt?p=505,upgrade-fbs_ddbcms_dk.search.ting.*,1,1536x864,24,1,1567420480920,2,1534x126,0&tz=2&eid=2156654855795232235&one=0&fns=0&geid=815665485570045126951269&ct=Kampagne&la=da&ck62=Kampagne%20plus&ck63=101&pu=https%3A%2F%2Fupgrade-fbs.ddbcms.dk%2Fsearch%2Fting%2F*%3F&eor=1 HTTP/1.1

Piotr?
Modtager du, hvad du skal?

#75 Updated by Tue Gaston 3 months ago

  • Assignee changed from Tue Gaston to Piotr Birketoft

#76 Updated by Piotr Birketoft 3 months ago

Hej Tue,

Ja og nej.
På siden https://upgrade-fbs.ddbcms.dk/ modtager jeg følgende parametere ved load:

p 505,upgrade-fbs_ddbcms_dk.,1,1920x1080,24,1,1567423824397,2,1413x233,0
tz 2
eid 2154296400267705299
one 0
fns 0
ct Kampagne
la en
ck62 Kampagne plus
ck63 101
pu https://upgrade-fbs.ddbcms.dk/
eor 1

 

ck62 er i WTK konfigurationen angivet til : KampagnePlus Titel

ck63 er i WTK konfigurationen angivet til : KampagnePlus ID

Vi ser den ikke her men ck64 KampagnePlus TAG

 

Så det er for så vidt fint nok.

Udfordringen er, at jeg modtager kaldet 2 gange:

Det er selvfølgeligt lidt problematisk?

 

- Piotr

 

#77 Updated by Tue Gaston 3 months ago

Piotr Birketoft wrote:

Udfordringen er, at jeg modtager kaldet 2 gange:

Det er selvfølgeligt lidt problematisk?

@Simon?

#78 Updated by Tue Gaston 3 months ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Piotr Birketoft to Simon Holt

#79 Updated by Simon Holt 3 months ago

Det ser altså ud til, at den kun sender event en gang. Kan du ikke lige prøve at teste igen @Piotr?

Jeg har sat debug-mode til på https://ddbcms.vejlebib.dk/search/ting/lydb%C3%B8ger. Så hvis du åbner console i dev tools, kan man også se, at den kun sender event en gang. Se vedhæftning. Hvis den sendte 2 gange ville der også komme dobbelt output i console. Så det må altså være noget andet der driller.

#80 Updated by Simon Holt 3 months ago

  • Assignee changed from Simon Holt to Piotr Birketoft

#81 Updated by Piotr Birketoft 3 months ago

Hej Simon,

Jeg kan bekræfte at du har ret, kampagne kaldet kommer nu kun 1 gang.

Jeg har nu kun 1 onclick kald med kampagnen i. Det andet er selve sidevisningen. (Se nedenfor).

 

- Piotr

 

Kald 1:

 

Kald 2:

 

#82 Updated by Tue Gaston 3 months ago

Piotr Birketoft wrote:

Hej Simon,

Jeg kan bekræfte at du har ret, kampagne kaldet kommer nu kun 1 gang.

Jeg har nu kun 1 onclick kald med kampagnen i. Det andet er selve sidevisningen. (Se nedenfor).

 

- Piotr

 

Kald 1:

 

Kald 2:

 

Kaldet sendes nu kun én gang - men er det kun konstateret på vejlebib? Eller er det også tilfældet på upgrade?
Og er der behov for at dykke ned i, hvorfor kaldet blev sendt dobbelt i første omgang med henblik på at sikre, at det ikke sker igen?
Eller har vi tillid til at det virker som det skal nu?
Kan dette issue lukkes nu?
Eller?

#83 Updated by Piotr Birketoft 3 months ago


Hej Tue,

 

Tak for du er vågen. Det var godt når jeg ikke er det.

 

Det virker på vejlebib, der kommer 2 kald, men de er forskellige, og det er kald nr 2 som har kampagne.

På Upgrade har vi stadig problemet, der kommer 2 ens kald.

 

- Piotr

#84 Updated by Simon Holt 3 months ago

Hvilken side på upgrade udløser to events? Prøv evt at slå debug mode til under /admin/config/ding/webtrekk og se hvad den outputter i Javascript console.

#85 Updated by Tue Gaston 3 months ago

Efter dialog med Piotr og Simon er det nu afdækket, at kaldet sendes to gange, fordi kampagnen vises to gange.
Alt er derfor som det skal være.

Dette issue er godkendt, og lukkes hermed.

#86 Updated by Tue Gaston 3 months ago

  • Status changed from Reviewed - Needs info/rework to Resolved (tag version)

#87 Updated by Tue Gaston 3 months ago

  • Assignee changed from Piotr Birketoft to Gitte Barlach

Also available in: Atom PDF