Project

General

Profile

Project activity #29141

closed

Refactor code GZAC

Added by Erik Hoogland 9 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
P4 Normal
Assignee:
Category:
-
Target version:
Start date:
15/08/2024
Due date:
% Done:

100%

Estimated time:

Description

In Tezza-gzac zitten een aantal punten die verbeterd kunnen worden, voor onderhoudbaarheid van de code.
Een aantal van deze zijn:
  • Hardcoded veldnamen voor proces variabelen, dit kan via static fields
    • Dit zit voornamelijk in de Beans
  • Hardcoded verwijzing nar email template in TezzaProcessBean, dit moet via plugin configuratie
  • Hardcoded (wel static) verwijzing naar standaard Tezza project. Dit moet via plugin configuratie
    • mogelijk ondersteuning voor meerdere standaard processen, is een grotere refactor
  • Ondersteuning voor meerdere jar's meenemen bij maken van een image
    • Door het startup commando te veranderen van run.sh naar *.jar
    • controleren of resources het correcte classpath krijgen, met nadruk op extra dossiers
    • controleren of Java Beans correct werken

Deze lijst kan nog verder worden uitgebreid.

Actions #1

Updated by Erik Hoogland 7 months ago

  • Description updated (diff)

AlfrescoService is al gerealiseerd via een ander ticket, wordt uit de lijst verwijderd

Actions #3

Updated by Erik Hoogland 7 months ago

  • Status changed from Backlog to In Progress
  • Assignee set to Erik Hoogland
  • Target version changed from Contezza Development Backlog to Contezza Development 2024.10

Het tezza-gzac project is zo aangepast, dat het naast een image ook een artifact zonder repackaged libs aanmaakt.
In een branch van het tezza-workspace project is een tezza-workspace-gzac module toegevoegd.
Deze build en repackaged een versie van gzac, met tezza-gzac-backend als voorbeeld. Het bevat ook een dummy project en Java bean om de correcte werking te testen.

Nog uit te zoeken is hoe we dit in de build straat van tezza-workspace willen zetten.

Actions #4

Updated by Erik Hoogland 7 months ago

Hardcoded verwijzing nar email template in TezzaProcessBean omgezet in property van tezza plugin

Actions #5

Updated by Erik Hoogland 7 months ago

Alle refactor punten zijn doorgevoerd in tezza-gzac.

Enige openstaande punt is de implementatie in tezza-workspace. Hier zijn twee openstaande punten:
  • Er is een build conflict met gzac. Deze heeft Java 17 nodig, de workspace wordt op 11 gebouwd
    • Dit gaat vooral om de Springboot repackage, deze heeft versie 17 nodig, en pakt de setting van de toolchains niet over
  • Er moet worden bepaald waar we de packaging van gzac gaan plaatsen
Actions #6

Updated by Erik Hoogland 7 months ago

  • Assignee changed from Erik Hoogland to Rick de Rooij

@Rick de Rooij De connectie tussen nginx en frontend is gefixed, en de livereload werkt nu ook.

Er zit nog een bug in de reload, waarbij er het inladen van een bean iets fout gaat. Maar ik heb ook gezien dat dit ook bij tezza-gzac fout gaat, dus mogelijk een probleem om samen met Ritense uit te zoeken, in een nieuw ticket.

Kan jij dit nalopen, en de benodigde zaken voor deployment toevoegen/corrigeren?

Actions #7

Updated by Tahir Malik 6 months ago

  • Project changed from 287 to Tezza
  • Target version changed from Contezza Development 2024.10 to Tezza 2024.10
  • Parent task changed from #26049 to #29845
Actions #8

Updated by Erik Hoogland 6 months ago

De versie van gzac is verhoogd naar 0.6.6-SNAPSHOT, en het is nu ook opgenomen in de snapshot build van gzac in Nexus. Er kan nu ook een build worden gemaakt die voor dev-tezza gaat werken

Actions #10

Updated by Rick de Rooij 6 months ago

Wijzigingen zit in branch: release/24.20. Zal vandaag nog niet worden gereleased. Moet nog wat meer testen.

Actions #11

Updated by Olav Allema 6 months ago

  • Target version changed from Tezza 2024.10 to Tezza 2024.11
Actions #12

Updated by Rick de Rooij 6 months ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Verwerkt in Tezza Workspace 24.20.

Also available in: Atom PDF