Project

General

Profile

Bug #3037

Biblioteket app'en - Smartbanner

Added by Gitte Barlach almost 3 years ago. Updated about 2 years ago.

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

Description

Via modulet easyddb_smartbanner kan DDC CMS konfigureres således at der tilføjes et lille banner når sitet indlæses fra forskellige mobile platforme. Banneret anbefaler at downloade App.

Foruden inkluderingen i Core skal modulet:

  • omdøbes til: “Ding Smartbanner”
  • eksporteres med konfigurede værdier (dog ikke aktiveres som default)
file (8).png (326 KB) file (8).png Vanilla-fbs i min iPhone (safaribrowser). Der vises ikke noget smartbanner Gitte Barlach, 04/12/2018 02:30 PM

History

#1 Updated by Gitte Barlach almost 3 years ago

  • Description updated (diff)

#2 Updated by Rolf Madsen almost 3 years ago

  • Target version set to Release 27 - Bugfixes (Inlead)

#3 Updated by Christel Krabbenhøft almost 3 years ago

  • Subject changed from Biblioteket app´en - smartbanner to Biblioteket app'en - smartbanner
  • Status changed from Ready for development to Needs design decision
  • Assignee changed from Martin Cording to Rolf Madsen
  • Target version changed from Release 27 - Bugfixes (Inlead) to Release 29-2 - Bugfixes (7.x-4.5.0)

#4 Updated by Martin Cording over 2 years ago

#5 Updated by Rolf Madsen over 2 years ago

  • Priority changed from Normal to Immediate

#6 Updated by Martin Cording over 2 years ago

  • Subject changed from Biblioteket app'en - smartbanner to Biblioteket app'en - Smartbanner
  • Description updated (diff)
  • Status changed from Needs design decision to Ready for development
  • Assignee changed from Rolf Madsen to Martin Cording
  • Priority changed from Immediate to High
  • Target version changed from Release 29-2 - Bugfixes (7.x-4.5.0) to Release 29-2 - Bugfixes (Inlead)

#7 Updated by Christel Krabbenhøft over 2 years ago

  • Target version changed from Release 29-2 - Bugfixes (Inlead) to Release 28 - Søge abstraktionslag (SAL) - (7.x-4.3.1)

#8 Updated by Gitte Barlach over 2 years ago

Hej Martin 
Opdaterer du sagen med et PR ?

#9 Updated by Martin Cording over 2 years ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Martin Cording to Christel Krabbenhøft

Vi afventer information omkring standard konfigurationen.
 

#10 Updated by Christel Krabbenhøft over 2 years ago

  • Assignee changed from Christel Krabbenhøft to Martin Cording

Hej Martin. Har I fået den nødvendige information fra Christian Sisseck?

#11 Updated by Martin Cording over 2 years ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Martin Cording to Gitte Barlach

#12 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#13 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Jesper Kristensen to Kasper Garnæs

#14 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Kasper Garnæs to Jesper Kristensen

#15 Updated by Jesper Kristensen over 2 years ago

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

Der var lige et par mindre sprøgsmål, men koden ser ellers fin ud.

#16 Updated by Christel Krabbenhøft over 2 years ago

Hej Martin. Har I mulighed for at vende retur på denne i dag?

#17 Updated by Martin Cording over 2 years ago

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

PR opdateret.

#18 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#19 Updated by Jesper Kristensen over 2 years ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jesper Kristensen to Gitte Barlach

Reviewed og afventer release

#20 Updated by Gitte Barlach over 2 years ago

  • Status changed from Reviewed to Technical test

Denne er merged.

#21 Updated by Michael W. Christoffersen over 2 years ago

  • Description updated (diff)

Modulet er aktiveret på upgrade, så den kan umiddelbart ikke godkendes.

#22 Updated by Gitte Barlach over 2 years ago

Det er ikke krævet at modulet skal aktiveret som default, da det er op til det enkelte bibliotek om de vil reklamere for app´en på deres hjemmeside. 

#23 Updated by Rolf Madsen over 2 years ago

Det har været et krav at modulet som default IKKE var aktiveret, netop for at bibliotekerne selv kunne aktivere det hvis de ønskede at føre trafik over på APP'en.

#24 Updated by Martin Cording over 2 years ago

Modulet aktiveres ikke som default med vores ændringer. Det er nok fordi at det har været aktiveret på upgrade inden opdatering.

Prøv at teste på vanilla.

#25 Updated by Michael W. Christoffersen over 2 years ago

  • Assignee changed from Gitte Barlach to Martin Cording

Der er det også aktiveret.

#26 Updated by Gitte Barlach over 2 years ago

@Michael: jeg testede i går aftes på vanilla og aktiverede derfor modulet. Det var ikke aktiveret som default, så det er som det sakl være.

Jeg har checket at konfigurationerne er sat. Så det ser også fint ud. Mangler lige en endelig bekræftelse fra Redia, og så kan vi godkende sagen her. 

#27 Updated by Gitte Barlach over 2 years ago

Har gentestet denne på vanilla-fbs. Jeg har slået modulet ding_app_smartbanner til, fået Redia til at bekræfte at default-konfigurationerne er rigtige. Jeg har uploaded app-ikonet. Jeg har slettet cashe. 
Trods dette får jeg desværre ikke vist smartbanneret når jeg går ind på sitet via min iPhone. Se vedlagte.

#28 Updated by Rolf Madsen over 2 years ago

Jeg slog tidligere på dagen smartbanner til på upgrade (tror jeg det var) og åbnede siden på min android 8.0 mobil og der blev smartbanneret vist korrekt.

#29 Updated by Rolf Madsen about 2 years ago

  • Status changed from Technical test to Resolved (tag version)
  • Assignee deleted (Martin Cording)

Also available in: Atom PDF