Project

General

Profile

Feature #3076

Udvikling af ding_webtrekk modul

Added by Rolf Madsen almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved (tag version)
Priority:
Immediate
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Administration - Systemkonfiguration, Driftsvedligehold - Refaktorering (Opdatering af kodebasen)

Description

På baggrund af Oracle's opkøb af Webtrend har KPI index (Visionize) valgt Webtrekk som nyt værktøj.

DDB ønsker at foretage skiftet inden 1. januar 2018 for at sikre datakoknsistens.

Derfor har vi behov for udvikling af:

  1. et nyt Ding_webtrekk modul til erstatning for det eksisterende Webtrends modul (https://github.com/ding2/ding2/tree/master/modules/ding_webtrends)
  2. Scripts til tracking af AJAX kald ved Facetbrowseren, "Se online knappen" og "Redigér knappen" via Webtrekk's Tag integration funktion. Se #1748

History

#1 Updated by Jesper Kristensen almost 2 years ago


Modulet ligget i github.com/ding2/ding2 under modules webtrends. @rolf det du linker til er gamlet og ikke brugt i årvis.

Vi skal osse have søgninger og hits med over.

#2 Updated by Rolf Madsen almost 2 years ago

  • Description updated (diff)

#3 Updated by Rolf Madsen almost 2 years ago

Tak for heads-up Jesper!

Linket er nu rettet til https://github.com/ding2/ding2/tree/master/modules/ding_webtrends

#4 Updated by Martin Cording almost 2 years ago

  • Assignee changed from Christel Krabbenhøft to Martin Cording

#5 Updated by Rolf Madsen almost 2 years ago

  • Target version changed from Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1) to DDB CMS 2017 1.2 opgradering - Webtrekk (7.x-4.1.0)

Webtrekk modul + click-event scripts

DDB CMS 2016 2. opgradering (7.x-3.0.2)

Webmasterbiblioteker skal kunne installere Webtrekk modulet, selv om de ikke opgraderer til DDB CMS 2017 1. opgradering.

Derfor vil vi gerne acceptere at:

  1. der laves et stand-alone Webtrekk modul som de kan installere i eksistrende DDB CMS 2016 2. opgradering miljøer.
  2. modulet IKKE inkluderer click-event scripts, hvis det viser sig at øge kompleksiteten unødigt.
     

DDB CMS 2017 1. opgradering (7.x-4.0.2)

Biblioteker der allerede har opgraderet til 2017 1. opgradering skal kunne opgradere til DDB CMS 2017 1. opgradering + Webtrekk scripts (7.x-4.1.0) som indeholder:

  1. Webtrekk modulet
  2. click-event scripts

#6 Updated by Martin Cording almost 2 years ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Martin Cording to Gitte Barlach

#7 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs
  • Priority changed from Normal to Immediate

#8 Updated by Kasper Garnæs almost 2 years ago

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

Reviewed. Jeg har en række spørgsmål og kommentarer.

#9 Updated by Martin Cording almost 2 years ago

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

Fixed

#10 Updated by Simon Holt almost 2 years ago

Ser fint ud :)

Hæftede mig dog ved, at den kun sætter OSS parametre, hvis der er resultater: https://github.com/ding2/ding2/pull/916/files#diff-33df4c1b1ac2ad0d1a03e79ea38959e6R84

Mister vi ikke viden om nul-hit søgninger eller er det hensigten det skal trackes på en anden måde?

#11 Updated by Kasper Garnæs almost 2 years ago

  • Status changed from Needs code review to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Godkendt og merged.

Bemærk: Modulet bliver pt. ikke slået til per default. Det skal gøres manuelt.

#12 Updated by Martin Cording almost 2 years ago

@Simon: Korrekt - skal dette check udelades?

#13 Updated by Simon Holt almost 2 years ago

Arrg var ikke hurtig nok. Havde overset dit svar, Martin.

Tjekket bør nok udelades, da det ellers vil være svært at se hvilke søgninger, der giver nul-hits for brugerne.

Er ikke klar over om I fjernede tjekket, men ser det ikke som noget stort problem.

#14 Updated by Christel Krabbenhøft almost 2 years ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Martin Cording

Hej Martin. Jf. Simons kommentar - er I så ikke rare at lave rettelsen asap?

#15 Updated by Martin Cording almost 2 years ago

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

#16 Updated by Kasper Garnæs almost 2 years ago

  • Status changed from Needs code review to Technical test

Reviewed, godkendt og merged.

Simon: Godt spottet. Tak for kommentaren.

Martin: Tak for hurtig opfølgning.

#17 Updated by Simon Holt almost 2 years ago

  • Status changed from Technical test to Needs code review

Havde lige et par ydeligere rettelser efter installation på vejlebib.dk:

https://github.com/ding2/ding2/pull/928

Inkluderer rettelsen fra PR ovenfor.

#18 Updated by Christel Krabbenhøft almost 2 years ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#19 Updated by Kasper Garnæs almost 2 years ago

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

Reviewed. Jeg har et par opfølgnende spørgsmål.

#20 Updated by Simon Holt almost 2 years ago

@Kasper har svaret inde på Github.

#21 Updated by Christel Krabbenhøft almost 2 years ago

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

#22 Updated by Simon Holt almost 2 years ago

@Kasper Har opdateret PR med de foreslåede ændringer.

Har også smidt de seneste ændringer på vejlebib.dk og det ser ud til at spille!

Lige en enkelt bemærkning: ved tomme søgeresultater stripper _ding_webtrekk_set_data_layer_values() værdien der viser antal søgeresultater, da den er 0 i disse tilfælde. Spørgsmålet er hvordan webtrekk håndtere dette? 

#23 Updated by Martin Cording almost 2 years ago

@Simon: Er dine ændringer testet i 2016-2? Når alt er færdig skal det nemlig overføres til at standalone repository, som Webmaster-biblioteker kan hente og uploade. Alternativt skal Core team levere en .zip-fil til Rolf.

#24 Updated by Simon Holt almost 2 years ago

Er rimelig sikker på det spiller der også, da der er tale om simple ændringer. Men smider en 16-2 op i morgen og tester for en sikkerheds skyld.

Er det ikke nemmest for alle parter bare at trække modulet ud og distribuere som zip-fil?

#25 Updated by Simon Holt almost 2 years ago

Har testet i 16-2 (3.0.2):

- Scriptet bliver indsat korrekt når man har konfigueret i backend!

- OSS parametere indsættes korrekt ved søgning i materialer og hjemmeside.

Lige en enkelt bemærkning: ved tomme søgeresultater stripper _ding_webtrekk_set_data_layer_values() værdien der viser antal søgeresultater, da den er 0 i disse tilfælde. Spørgsmålet er hvordan webtrekk håndtere dette? 

Har spurgt Piotr om dette og det lader ikke til at være problem at håndtere dette i webtrekk.

#26 Updated by Gitte Barlach almost 2 years ago

Kan du dermed godkende modulet, Simon?

#27 Updated by Gitte Barlach almost 2 years ago

hvordan får vi det testet på en 17-2 installation? Altså mere end bare at modulet "er der" og kan slåes til"???

#28 Updated by Michael W. Christoffersen almost 2 years ago

Du mener vel 17.1?

#29 Updated by Simon Holt almost 2 years ago

Ja, jeg kan godkende det.

Kører med det på vejlebib.dk nu, hvilket svarer til en 4.0.2. Har også adgang til webtrekk kontrolpanel og kan se den sender data.

#30 Updated by Gitte Barlach almost 2 years ago

  • Status changed from Needs code review to Resolved (tag version)
  • Assignee changed from Kasper Garnæs to Simon Holt

ja, 17-1 selvfølgelig!
Dejlig du kan godkende - ændrer derfor status til 'resolved'

Also available in: Atom PDF