Projekt

Generelt

Profil

Enhancement #1551

Opdater eksporteret konfiguration

Tilføjet af Kasper Garnæs for mere end 3 år siden. Opdateret for 9 måneder siden.

Status:
Ready for development
Prioritet:
Urgent
Tildelt til:
Anslået tid:
URL med eksempel:
Kategorier:
Driftsvedligehold - Refaktorering (Opdatering af kodebasen)

Beskrivelse

Martin Cording, Inlead har gjort opmærksom på at modulopdateringer har medført at vores konfiguration skal opdateres, så den er opdateret tilsvarende. Dermed bør pull requests med ændringer ad konfiguration fremadrettet kun indeholde de relevante ændringer.

Dette bør gøres i forbindelse med næste release.

Historik

#2 Opdateret af Kasper Garnæs for mere end 3 år siden

Det er vigtigt at bemærke at denne opgave bør laves på en vanilla installation af DDB CMS - ikke ud fra et kørende produktionssite.

#3 Opdateret af Rolf Madsen for mere end 3 år siden

  • Status ændret fra New til Ready for development
  • Prioritet ændret fra Normal til Urgent
  • Udgave sat til DDB CMS 2016 1. opgradering

#4 Opdateret af Martin Cording for mere end 3 år siden

Og den burde laves umiddelbart efter næste release :)

#5 Opdateret af Rolf Madsen for cirka 3 år siden

  • Udgave ændret fra DDB CMS 2016 1. opgradering til 42

#6 Opdateret af Rolf Madsen for cirka 3 år siden

  • Tildelt til ændret fra Rolf Madsen til Laura Holm

#7 Opdateret af Rolf Madsen for cirka 3 år siden

  • Tildelt til slettet (Laura Holm)

#8 Opdateret af Rolf Madsen for cirka 3 år siden

  • Udgave ændret fra 42 til DDB CMS 2016 1. opgradering

#9 Opdateret af Rolf Madsen for cirka 3 år siden

  • Udgave ændret fra DDB CMS 2016 1. opgradering til DDB CMS 2016 2. opgradering (Inlead sprintbacklog)

#10 Opdateret af Martin Cording for cirka 3 år siden

@Rolf: Denne opgave burde ligge "udenfor" de oprindelige udviklingsrunder og laves lige efter 2.4.x (alternativt 2.5.x) er released.

#11 Opdateret af Martin Cording for cirka 3 år siden

  • Udgave ændret fra DDB CMS 2016 2. opgradering (Inlead sprintbacklog) til DDB CMS 2016 2. opgradering

#12 Opdateret af Rolf Madsen for cirka 3 år siden

  • Udgave ændret fra DDB CMS 2016 2. opgradering til DDB CMS 2016 2. opgradering (DBC sprintbacklog)

#13 Opdateret af Rolf Madsen for næsten 3 år siden

  • Kategorier Driftsvedligehold - Refaktorering (Opdatering af kodebasen) tilføjet

#14 Opdateret af per johansen for næsten 3 år siden

  • Status ændret fra Ready for development til Development
  • Tildelt til sat til per johansen

planen er helt bevidstløst at genskabe features fra en frisk installation.

#15 Opdateret af per johansen for næsten 3 år siden

  • Status ændret fra Development til Needs code review
  • Tildelt til ændret fra per johansen til Kasper Garnæs

Når i siger konfiguration går jeg udfra at i mener indstillinger i features. Hvis jeg kigger på et vanilla-site (https://vanilla-alma.ddbcms.dk/) er disse features "tilsidesat";
bpi
ding_content
ding_frontend
ding_groups

ding_content er tilsidesat pga den hook der bruges i ding_content/ding_content.file_default_display.inc se fx dether issue:https://www.drupal.org/node/2104193, der er en patch til features, der gør at den ikke vises som "tilsidesat", men den anbefalede løsning (som er brugt i Media modulet) er at flytte konfigurationen til install

ændringerne i ding_frontend og ding_groups er små og ligegyldige

summa summarum har jeg lavet et pull-request på ændringerne i bpi
https://github.com/ding2/ding2/pull/205

#16 Opdateret af Kasper Garnæs for næsten 3 år siden

  • Status ændret fra Needs code review til Reviewed - Needs info/rework
  • Tildelt til ændret fra Kasper Garnæs til per johansen

Pull requestet indholder ingen kodeændringer pt. så der er intet at reviewe.

Generelt kan jeg ikke få det til at stemme at Martin Cording melder at pull requests med konfigurationsændringer indholder irrelevante ændringer pga. modulopdateringer, mens der ifølge Per nærmest ikke er ændringer.

> ding_content er tilsidesat pga den hook der bruges i ding_content/ding_content.file_default_display.inc se fx dether issue:https://www.drupal.org/node/2104193, der er en patch til features, der gør at den ikke vises som "tilsidesat", men den anbefalede løsning (som er brugt i Media modulet) er at flytte konfigurationen til install

Burde vi i så fald ikke flytte vores konfiguration til install også?

> ændringerne i ding_frontend og ding_groups er små og ligegyldige

Hvilke ændringer er der tale om? Generelt handler dette issue jo om at undgå at features er unødvendigt overriden, så derfor tænker jeg som udgangspunkt at vi skal have disse ændringer med hvor små de så end måtte være.

#17 Opdateret af per johansen for næsten 3 år siden

  • Tildelt til ændret fra per johansen til Martin Cording

Måske Martin kan være mere specifik på hvad der er for ændringer der er tale om - som sagt kan jeg ikke rigtig finde nogen

#18 Opdateret af Martin Cording for næsten 3 år siden

  • Tildelt til ændret fra Martin Cording til per johansen

Jeg har ikke lige noget konkret vedr. dette, men det gælder primært Views og Panels som er eksporteret - hvorefter versionen af disse moduler er opdateret.
Efter opdatering af contrib modulet, er der ændrede måder at strukturere den eksporterede kode på.

Det vil altså sige at laver man en lille ændring til f.eks. et View, så er diff'en kæmpe stor fordi at det er ønsket at man anvender den nye/opdaterede formatering/struktur.
Du kunne grep'e efter features i kodebasen, eksportere alt hvad der ligger og se diffen.

#19 Opdateret af Rolf Madsen for mere end 2 år siden

  • Tildelt til ændret fra per johansen til Laura Holm

#20 Opdateret af per johansen for mere end 2 år siden

  • Status ændret fra Reviewed - Needs info/rework til Need more info
  • Tildelt til ændret fra Laura Holm til Martin Cording

jamen altså -

Ville det ikke være bedre at tage dem som de kommer.

Hvis der bliver lavet en ændring i én feature må core jo reviewe den éne ændring - Istedet for som du foreslår, hvor diffen potentielt bliver gigantisk og HELT umulig at reviewe på fornuftig vis ...

og hvad hvis der opstår fejl - hvordan ville vi kunne debugge den gigantiske diff - det ville være nemmere med én ad gangen, også hvis vi evt. skal rulle tilbage.

#21 Opdateret af Martin Cording for mere end 2 år siden

  • Tildelt til ændret fra Martin Cording til Kasper Garnæs

#22 Opdateret af Gitte Barlach for mere end 2 år siden

Kan denne vente til næste release?

#23 Opdateret af Kasper Garnæs for mere end 2 år siden

  • Status ændret fra Need more info til Needs analysis
  • Tildelt til ændret fra Kasper Garnæs til Gitte Barlach

Jo. Det må den.

#24 Opdateret af Kasper Garnæs for mere end 2 år siden

  • Udgave ændret fra DDB CMS 2016 2. opgradering (DBC sprintbacklog) til DDB CMS 2017 1. opgradering (7.x-4.0.2)

#25 Opdateret af Kasper Garnæs for næsten 2 år siden

Jeg har netop kørt features update og får stadig en række ændringer, så jeg mener stadig denne er relevant.

Samtidig har issues så som #2304 og projekter der vil benytte Features Override i nyeste version behov for nyeste version af features.

Derfor vil jeg foreslå at man i samme forbindelse med denne ticket også opdaterer Features til seneste version, pt. 2.10.

#26 Opdateret af Rolf Madsen for næsten 2 år siden

  • Status ændret fra Needs analysis til Ready for development

#27 Opdateret af Kasper Garnæs for næsten 2 år siden

Ændringerne i #1551 kunne medtages i denne opgave.

#28 Opdateret af Rolf Madsen for næsten 2 år siden

  • Udgave ændret fra DDB CMS 2017 1. opgradering (7.x-4.0.2) til Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1)

#29 Opdateret af Gitte Barlach for mere end et år siden

  • Beskrivelse updated (diff)
  • Tildelt til ændret fra Gitte Barlach til Martin Cording

Hej Martin 

Har I mulighed for at kigge på denne?

#30 Opdateret af Martin Cording for mere end et år siden

  • Tildelt til ændret fra Martin Cording til Christel Krabbenhøft

Jeg er usikker på hvad der skal ske her.

Skal alle features gen-eksporteres, eller skal features modulet "blot" opgraderes til 2.10?

#31 Opdateret af Rolf Madsen for mere end et år siden

  • Status ændret fra Ready for development til Need more info
  • Tildelt til ændret fra Christel Krabbenhøft til Kasper Garnæs

@Kasper & @Per har I nogen input her?

Det er et meget teknisk issue så jeg kan ikke byde ind med noget ...

#32 Opdateret af Rolf Madsen for cirka et år siden

  • Udgave ændret fra Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1) til Release 29-2 - Bugfixes (7.x-4.5.0)

#33 Opdateret af Rolf Madsen for cirka et år siden

  • Status ændret fra Need more info til Needs analysis
  • Tildelt til ændret fra Kasper Garnæs til Gitte Barlach

@Gitte, vil du ikke booke Core team til at tage et kig på den her?

#34 Opdateret af Rolf Madsen for cirka et år siden

  • Tildelt til ændret fra Gitte Barlach til Kasper Garnæs
  • Udgave ændret fra Release 29-2 - Bugfixes (7.x-4.5.0) til Release 29-2 - Bugfixes (Inlead)

#35 Opdateret af Kasper Garnæs for 11 måneder siden

  • Status ændret fra Needs analysis til Ready for development
  • Tildelt til ændret fra Kasper Garnæs til Martin Cording

Skal alle features gen-eksporteres, eller skal features modulet "blot" opgraderes til 2.10?

I mine øjne begge dele.

#36 Opdateret af Rolf Madsen for 9 måneder siden

  • Udgave ændret fra Release 29-2 - Bugfixes (Inlead) til Release 33 - Bugfixes

Eksporter til Atom PDF