Project

General

Profile

Bug #1744

Tilføj flere forskellige karruseller på forsiden og sektionssider (Editorial base)

Added by Thomas Hansen over 4 years ago. Updated 10 months ago.

Status:
Resolved (tag version)
Priority:
Urgent
Estimated time:
URL med eksempel:
Kategorier:
Driftsvedligehold - Manual (udarbejdes før udrulning), Inspiration - Karruseller

Description

Når jeg klikker "Fjern", så AJAX reloader den, men den har ikke fjernet tab'en men kun fjernet titlen (hvilket er nemt at overse). Man skal gemme før den forsvinder helt.

Formularen bruger heller ikke Drupals add-more functionalitet, og vægtningen er også underlig: http://platform.dandigbib.org/issues/269

Den trænger til en kærlig hånd. Man kunne evt. slå det sammen med at gøre det muligt at tilføje flere karruseller.

Problemstilling

Det er kun muligt at tilføje én forsidekarrusel på forsiden og sektionssider (Editorial base).

Biblioteket app'en har en afhængighed til DDB CMS da den benytter /ding/variables til at generere en forsidekarrusel.

Formål

Bibliotekerne skal have mulighed for at tilføje forskellige karruseller så de kan bruges aktivt i formidlingsarbejdet og afspejle det emne der formidles.

Løsningsforslag

Flere forsidekarruseller

  • Forsidekarruseller skal kunne tilføjes som panes og indsættes på forsiden og sektionsforsider (Editorial base) med samme funktionalitet som den nuværende forsidekarrusel.
  • Faneblade skal kunne slettes jævnfør ovenstående tekst om "klikker "Fjern", så AJAX reloader den, men den har ikke fjernet tab'en".
  • Lokal administrator og Local editor rollerne skal alene have adgang til at indsætte nye elementer på forsiden og sektionssiderne.

NB. følgende issues tilføjer ny funktionalitet til de nye forsidekarruseller:

Biblioteket app'en og ding/variables

Under https://upgrade-fbs.ddbcms.dk/ding/variables skal DDB CMS fortsat kunne udlevere data:

  1. fra de Forsidekarruseller der er oprettet på forsiden
  2. i nedenstående format som Biblioteket app'en kan benytte
{  
   variables:{  
      opensearch_search_profile:"DDB Test",
      ting_search_results_per_page:"",
      ting_search_result_message_limit:"",
      opensearch_url:"https://opensearch.addi.dk/b3.5_5.0/",
      opensearch_sort_default:"",
      opensearch_ranking_custom:"",
      opensearch_ranking_fields:"",
      opensearch_boost_fields:"",
      ting_agency:"725300"
   },
   carousels:[  
      {  
         title:"Børn",
         subtitle:"Nye bøger til de små",
         query:"870970-basis:50612910 OR 870970-basis:29981299 OR 870970-basis:29967326 OR 870970-basis:50561488 OR 870970-basis:29618518 OR 870970-basis:50530671 OR 870970-basis:50577384 OR 870970-basis:50561445 OR 870970-basis:50587177 OR 870970-basis:50618404 "
      },
      {  
         title:"Unge",
         subtitle:"Nye bøger til unge",
         query:"870970-basis:29983739 OR 870970-basis:29983585 OR 870970-basis:29989702 OR 870970-basis:29997829 OR 870970-basis:29997918 OR 870970-basis:29986215 OR 870970-basis:50532615 OR 870970-basis:50613038 OR 870970-basis:50618374"
      }
   ]
}
Skærmbillede 2018-12-11 15.55.48.png (101 KB) Skærmbillede 2018-12-11 15.55.48.png Konfigurationen /admin/config/ting/ting_search_carousel på stable Gitte Barlach, 12/11/2018 03:56 PM
Skærmbillede 2018-12-11 15.54.51.png (884 KB) Skærmbillede 2018-12-11 15.54.51.png Det site vi opgraderer fra har 2 karuseller Gitte Barlach, 12/11/2018 03:56 PM

Related issues

Related to DDB CMS - Bug #3465: Forsidekarrusel materialer skal kunne sortéresResolved (tag version)
Related to DDB CMS - Enhancement #1353: Forsidekarussellen skal kunne indstilles til kun at vise materialer, der er hjemmeResolved (tag version)
Related to DDB CMS - Enhancement #3158: Forsidekarrusellen skal have mulighed for at filtrere materialerne med eller uden forsiderClosed
Related to DDB CMS - Enhancement #4115: /ding/variables skal kunne udlevere data efter #1744Resolved (tag version)

History

#1 Updated by Rolf Madsen over 4 years ago

  • Status changed from New to Ready for development
  • Target version set to 34

#2 Updated by Rolf Madsen about 4 years ago

  • Target version changed from 34 to Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1)

#3 Updated by Rolf Madsen almost 4 years ago

  • Assignee set to Martin Cording
  • Target version changed from Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1) to Release 27 - Bugfixes (Inlead)

#4 Updated by Martin Cording almost 4 years ago

Er det ikke muligt at tilføje flere karruseller nu? Blev vores issue med at flytte konfigurationen til panelete og ikke backenden, aldrig implementeret?

#5 Updated by Martin Cording almost 4 years ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Martin Cording to Gitte Barlach

Jeg har snakket om det tidligere i dette PR: https://github.com/ding2/ding2/pull/261

#6 Updated by Martin Cording over 3 years ago

  • Status changed from Need more info to Needs code review

#7 Updated by Gitte Barlach over 3 years ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#8 Updated by Thomas Hansen over 3 years ago

Gah, og jeg har sådan set lavet det samme for et par uger siden.

Men https://github.com/ding2/ding2/pull/508 ser godt ud, der er lige et par småting.

#9 Updated by Rolf Madsen over 3 years ago

Mangler vi lidt før code review qua Thomas' kommentar under https://github.com/ding2/ding2/pull/508#discussion_r112131670?

"This needs an update hook to migrate old variable to the new."

#10 Updated by Jørgen Nielsen almost 3 years ago

  • Description updated (diff)
  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Martin Cording

Koden mangler at blive opdateret ifm. Thomas' kommentarer. Den kunne også trænge til en rebase.

#11 Updated by Jesper Kristensen almost 3 years ago


vær opmærksom på at dette ikke helt er kombatibelt med https://platform.dandigbib.org/issues/1770 samt at omdøbning af "ting_carousel_search_queries" variablen vil få https://github.com/easySuite/ding_redia_variables til at stoppe med at virke.

#12 Updated by Rolf Madsen over 2 years ago

  • Target version changed from Release 27 - Bugfixes (Inlead) to Release 29-2 - Bugfixes (7.x-4.5.0)

#13 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

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

Bemærk at dette PR bør (rebases og) merges på samme tid: https://github.com/ding2/ding2/pull/1067

#14 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#15 Updated by Jesper Kristensen over 2 years ago

  • Status changed from Needs code review to Need more info
  • Assignee changed from Jesper Kristensen to Gitte Barlach

Koden ser fint ud og kan godkendes fra et teknisk synspunkt.

Men da det laver om på redaktørs workflow og flytte ændring af ting_search_carousel ind i panels skal dette forbi forretningslagt i DDB CMS først.

Title på dette issue skal måske laves om til: Moving ting_search_carousel into panels som er det PR'eret gør

 

#16 Updated by Rolf Madsen over 2 years ago

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

@Martin, kan du beskrive hvordan dit PR laver om på redaktør flowet?

Hvis det flytter konfigurationen af forsidekarrusellen fra /admin/config/ting/ting_search_carousel til panelet ville jeg ikke have noget problem med det, men jeg har nok brug for lidt screenshots og en kort beskrivelse.

Derudover ville vi være nødt til at lægge arbejde i manualen op til ændringen.

 

#17 Updated by Martin Cording over 2 years ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Martin Cording to Rolf Madsen

Det er korrekt at konfigurationen overflødiggøres og flyttes til panelet.

Sådan fungerer det inden PR: https://monosnap.com/file/RQaEcO1JbXdSJJiHhFl04mc3vUQ9j3
Sådan fungerer det efter PR: https://monosnap.com/file/UVy6viCgCvIUdYVmRN2a20o363mmCP

#18 Updated by Simon Holt over 2 years ago

For os er det lidt et problem, at den flyttes til panelet. Alle dem der skal redigere i forsidekarusellen, skal så samtidig også have adgang til at ændre hele forsiden eller adgang til page manager konfigurationen, som nok er lidt for teknisk.

Hvad er grunden til at den flyttes til panelet og kunne man ikke bare have den begge steder?

#19 Updated by Martin Cording over 2 years ago

Er det sådan at dem der har adgang til at rette forsidekarusellen ikke har adgang til forside panelet?

Det skyldes nok primært at der med #1604 skabes mulighed for at karusellen kan indsættes på alle under-forsider (sektioner/temaer).
Derfor vil det allerede nu være fordelagtigt at få dette inkluderet, så det ikke sker i samme release - hvor der dermed vil være den samme karussel på alle sider.

#20 Updated by Simon Holt over 2 years ago

Er det sådan at dem der har adgang til at rette forsidekarusellen ikke har adgang til forside panelet?

Ja, der er en permission "configure carousel": https://github.com/ding2/ding2/blob/master/modules/ting_search_carousel/ting_search_carousel.module#L62

Så det vil sige man får mulighed for at lave flere forskellige karuseller?

#21 Updated by Martin Cording over 2 years ago

Ja jeg forstår at man kan konfigurere sig ud af de tilladelser, men jeg tænker arbejdsmæssigt - er den "redaktør"/webmaster som styrer forsiden, ikke også den som bør kunne lave karusseler?

Ja, du kan indsætte ubegrænset antal karuseller på forsiden - og senere (med #1604) på forsiden samt alle underforsider.

#22 Updated by Simon Holt over 2 years ago

Ja jeg forstår at man kan konfigurere sig ud af de tilladelser, men jeg tænker arbejdsmæssigt - er den "redaktør"/webmaster som styrer forsiden, ikke også den som bør kunne lave karusseler?

Bevares :) Du må da give mig ret i, at der er rimelig stor forskel på at kunne ændre hele forsiden og bare tilføje nye søgninger til karusellen?

#23 Updated by Martin Cording over 2 years ago

Ja der er en forskel, men for mig er en forside-redaktør en forside-redaktør :)

#24 Updated by Simon Holt over 2 years ago

hehe.. vi er måske også lidt specielle her på vejlebib ;)

Vi bruger forsidekarusellen meget som et formidlingsredskab og derfor har mange bibliotekarer, der normalt ikke arbejder med hjemmesiden, fået adgang til konfigurationen.

Jeg synes det er lidt voldsomt, hvis de alle skal have adgang til at ændre forsiden, og til at konfiguere karusellen derfra.

#25 Updated by Rolf Madsen over 2 years ago

  • Related to Bug #3465: Forsidekarrusel materialer skal kunne sortéres added

#26 Updated by Rolf Madsen over 2 years ago

  • Target version changed from Release 29-2 - Bugfixes (7.x-4.5.0) to Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0)

#27 Updated by Rolf Madsen over 2 years ago

Uafklarede spørgsmål:

  • Hvilke roller kan i dag redigere forsidekarusellen?

  • Skal samme roller have adgang til forsiden og sektionssider så de kan indsætte karuseller?

#28 Updated by Thomas Hansen about 2 years ago

Eh, virker drag-n-drop til at styre rækkefølgen af søgninger?

Jeg er lige rendt ind i at tabledrag lader til at være flaky in popup vinduer, i både IPE og page manager.

#29 Updated by Rolf Madsen about 2 years ago

@Thomas, så vidt jeg kan se virker drag n' drop fint på /admin/config/ting/ting_search_carousel.

Show row heights vises og jeg trækker på kryds-pilen så Børn ligger øverst

Karrusellen gemmes med Børn øverst og det afspejles i frontend

 

#30 Updated by Rolf Madsen about 2 years ago

  • Subject changed from At slette tabs i forsidekarrusellen er forvirrende to Tilføj flere forskellige karruseller på forsiden og sektionssider (Editorial base)
  • Description updated (diff)
  • Status changed from Need more info to Ready for development
  • Assignee changed from Rolf Madsen to Christel Krabbenhøft
  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0) to Release 30 - BPI, Kampagneplus og Sektioner

#31 Updated by Rolf Madsen about 2 years ago

  • Description updated (diff)

#32 Updated by Rolf Madsen almost 2 years ago

  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner to Release 30 - BPI, Kampagneplus og Sektioner - (Reload)

#33 Updated by Rolf Madsen almost 2 years ago

  • Description updated (diff)

#34 Updated by Rolf Madsen almost 2 years ago

  • Description updated (diff)

#35 Updated by Rolf Madsen almost 2 years ago

  • Description updated (diff)

#36 Updated by Rolf Madsen almost 2 years ago

  • Related to Enhancement #1353: Forsidekarussellen skal kunne indstilles til kun at vise materialer, der er hjemme added

#37 Updated by Rolf Madsen almost 2 years ago

  • Related to Enhancement #3158: Forsidekarrusellen skal have mulighed for at filtrere materialerne med eller uden forsider added

#38 Updated by Rolf Madsen almost 2 years ago

  • Description updated (diff)

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

  • Priority changed from Normal to High

#40 Updated by Thomas Hansen almost 2 years ago

  • Status changed from Ready for development to Needs code review

Nyt PR: https://github.com/ding2/ding2/pull/1233

Inkluderer migrering af eksisterende karruseller.

 

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

  • Assignee changed from Christel Krabbenhøft to Gitte Barlach

#42 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#43 Updated by Jesper Kristensen almost 2 years ago

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

Reviewed med et par mindre kommentar der skal kigges på.

#44 Updated by Thomas Hansen almost 2 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review

Fixed.

 

#45 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Thomas Hansen to Jesper Kristensen

#46 Updated by Jesper Kristensen almost 2 years ago

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

Reviewed og afventer release

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

  • Status changed from Reviewed to Technical test

Merged.

#48 Updated by Stefan Søndervang almost 2 years ago

  • Assignee changed from Gitte Barlach to Stefan Søndervang

#49 Updated by Stefan Søndervang almost 2 years ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Stefan Søndervang to Thomas Hansen

Testet på fbs-upgrade.

Man kan lave flere karruseller på forsiden, men de eksisterende karruseller er ikke kommet med. Forsidekarrusellen var helt væk og jeg skulle lave nogle fra bunden.

Det er vigtigt at de kommer med.

#50 Updated by Thomas Hansen almost 2 years ago

Kan ikke reproducere. Jeg har lige checket ding2 pre-merge ud, oprettet en ekstra forsidekarrusel (havde ikke lige set den var der), konfigureret den på den gamle konfigurations side. Efter at have kørt updb er de der begge stadig.

 

#51 Updated by Gitte Barlach almost 2 years ago

Hej Thomas

Tak for feedback. Jeg tænker 2 ting:
a) jeg vil forhøre mig om Chr.V/DBC har kørt en updb på vores upgrade-fbs, som vi testede på. Det vil jeg tro, men jeg undersøger det. 
b) "den gamle konfigurationsside" er fjernet. Det skyldes at forsidekarrusellerne fra og med release 30 indsættes og konfigureres via Panels-in-place. Derfor er alle konfigurationer ift. forsiden blevet fjernet. Det er issue #3793
Så den ting kan måske godt gribe forstyrrende ind?
Endvidere har lagt mærke til at såfrem jeg fjerner en forside-karrusell via Panels-in-place, så fjerner man ikke bare karrusellen, men også indholdet af den. (Pkt. 3 i min kommentar 42 i #3269)

#52 Updated by Thomas Hansen almost 2 years ago

Nu checkede jeg specifikt en version af ding ud som var fra før det her blev merged, for netop at kunne konfigurere en "gammel" forside-karrusel, så de ændringer du snakker om har ikke været noget problem.

Hvad angår kommentar 42 i #3269, så, ja, når man sletter en pane, så ryger konfigurationen af den pane, og dermed "indholdet". Sådan er det jo.

Du kan også lige spørge dem om de rent faktisk har konfigureret "den gamle" karrusel før de opgraderede.

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

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Thomas Hansen to Gitte Barlach

#54 Updated by Rolf Madsen almost 2 years ago

  • Status changed from Need more info to Reviewed - Needs info/rework

#55 Updated by Christian Vandel almost 2 years ago

Updb kører *altid* på upgrade-fbs. Udgangspunktet er det site som hedder stable-fbs, som kører aktuel produktionsversion, altså p.t. version 4.5.0.

At ding_example-indholdet er dubleret på vanilla-fbs kan skyldes at der kører en feature revert all selve installationen. Det er dog ikke et issue  jeg før er stødt på...

 

#56 Updated by Gitte Barlach almost 2 years ago

"At ding_example-indholdet er dubleret på vanilla-fbs kan skyldes at der kører en feature revert all selve installationen. Det er dog ikke et issue  jeg før er stødt på..."  hører til en anden sag, #3989 

#57 Updated by Gitte Barlach almost 2 years ago

Kan du se på denne igen, Fini?

iflg. Christan så kører Updb *altid* på upgrade-fbs.
Han skriver endvidere at Udgangspunktet er det site som hedder stable-fbs, som kører aktuel produktionsversion, altså p.t. version 4.5.0.

Stable-fbs er endvidere konfigureret med 2 karuseller med indhold. Se vedlagte skærmdump.

Jeg håber det besvarer dine spørgsmål?

#58 Updated by Thomas Hansen over 1 year ago

@gitte

Jamen, så vil jeg da gerne have et DB dump fra stable-fbs, da jeg som sagt ikke kan reproducere med det jeg har.

#59 Updated by Gitte Barlach over 1 year ago

Jeg har oprettet en sag hos DBC kundeservice. (REQ0030013)

#60 Updated by Thomas Hansen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review

Problemet var ikke i migreringen. Problemet var at stable-fbs bruger standard forsiden, så den er ikke gemt i DB'en, hvilket gør at den ikke bliver migreret.

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

#61 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Thomas Hansen to Jesper Kristensen

#62 Updated by Gitte Barlach over 1 year ago

  • Priority changed from High to Urgent

#63 Updated by Jesper Kristensen over 1 year ago

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

Jeg er ikke helt med her med det sidste PR indsætte to karruseller på forsiden på alle installationeren. Det virker ikke til at det er det man ønsker, men der i mod skulle de måske være i example content modulet.

Eller vil disse to karruseller blive forced på alle installation hvergang der bliver lave en release og nogle reverter features? 

#64 Updated by Thomas Hansen over 1 year ago

@jesper

Well, forsiden er redigerbar med IPE, så enhver pillen ved forsiden vil jo udløse problemet, det er ikke specifikt for karousellen.

#65 Updated by Gitte Barlach over 1 year ago

Det vi egnetligt gerne ville opnå med den sidste rettelse var at bibliotekets eksisterende forsidekarruseller - hvis de har nogen - bliver migreret med over. Er det det, der er lavet her? 
Hvis det er for besværligt at håndtere rent kodemæssigt (vi har jo i en anden sag bedt om at få fjernet backend konfig til forsiden, herunder også forsidekarrusel) skal vi måske nærmere frafalde kravet og i stedet oplyse bibliotekerne om at de skal copy-paste deres karrusell-søgninger over i et workdokument og ud fra det oprette dem igen manuelt, når først de er ovre på rel. 30 ?

#66 Updated by Thomas Hansen over 1 year ago

@gitte

Migreringen for biblioteks-oprettede karruseller var på plads før sidste rettelse. Men på stable-fbs har ingen redigeret i forsiden, så den kører stadig på den der er defineret i koden i ding_frontend.

Problemet er at hvis der ikke er defineret karrusel i koden, så er der ikke nogen karrousel på forsiden per default, hvilket ser fesent ud.

#67 Updated by Gitte Barlach over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Technical test

ah, ok. Jeg sætter denne tech.test, og så opretter vi nogle karuseller på stable, og kontrollerer om de kommer med over på upgrade næste gang vi ruller en ny RC ud. 

#68 Updated by Gitte Barlach over 1 year ago

Jeg har nu oprettet flg. karuseller på stable-fbs:

Migreringstest 1
54471742 OR 54686056 OR 54871554 OR 54951620 OR 45929094

Migreringstest 2
54892594 OR 52823668 OR 46069374 OR 29577528 OR 54195745 

Migreringstest 3
54929250 OR 46010248 OR 46010299 OR 28298781 OR 53943616 OR 24795209

Disse bør jvf. Thomas migreres med over på upgrade-fbs når denne opgraderes med udgangspunkt i stable

#69 Updated by Gitte Barlach over 1 year ago

Testet på upgrade-fbs med 4.6.0-rc6
Ingen af de karruseller, jeg oprettede på stable-fbs, er kommet med over på upgrade-fbs.  

Elementer på forsiden:
Tabroll, kampagne, biblioteket foreslår materialer til dig, månedens bedst bedømte materialer, nyt fra biblioteket, arrangementer og udvalgte temaer. Men ingen forsidekarrusel. 

#70 Updated by Gitte Barlach over 1 year ago

  • Status changed from Technical test to Resolved (tag version)
  • Kategorier Driftsvedligehold - Manual (udarbejdes før udrulning) added

Godkender denne; vi må til gengæld huske at gøre bibliotekerne opmærksomme på at de skal gemme deres forsidekaruseller inden de opgraderer. Og derefter gen-indsætte dem på forsiden. 

#71 Updated by Stefan Søndervang over 1 year ago

  • Related to Enhancement #4115: /ding/variables skal kunne udlevere data efter #1744 added

#72 Updated by Thomas Hansen over 1 year ago

Hvad med at merge den før i lukker PRet?

#73 Updated by Rolf Madsen over 1 year ago

  • Status changed from Resolved (tag version) to Needs code review
  • Assignee changed from Gitte Barlach to Rolf Madsen
  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner - (Reload) to Release 32 - Bugfixes

@Thomas tak for din årvågenhed!

Jeg har bedt Core team tage et kig på den igen.

#74 Updated by Rolf Madsen over 1 year ago

  • Assignee changed from Rolf Madsen to Thomas Hansen

@Thomas jeg skal lige være helt klar på hvad ændringen i https://github.com/ding2/ding2/pull/1317/files gør.

Er det default indhold ved en ny installation eller er det default indhold når man tilføjer en karusel til forside eller sektionsside?

#75 Updated by Thomas Hansen over 1 year ago

@rolf

Ny installation, eller hvis de ikke har pillet ved forsiden endnu (nok de færreste).

 

#76 Updated by Rolf Madsen over 1 year ago

Tak for update Thomas.

Efter input fra Core team foretrækkes det at eksempel indhold:

  1. Ikke har afhængigheder til OpenSearch (fx. id'erne) og dansk.
  2. Placeres i Example Content modul.

Det er ikke opgaver vi ønsker at prioritere på nuværende tidspunkt.

Derfor fastholder vi lukning af https://github.com/ding2/ding2/pull/1317/

#77 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Thomas Hansen to Jørgen Nielsen

#78 Updated by Stefan Søndervang 10 months ago

  • Status changed from Needs code review to Resolved (tag version)
  • Target version changed from Release 32 - Bugfixes to Release 30 - BPI, Kampagneplus og Sektioner

Also available in: Atom PDF