GeefVolgendeAfTeDrukkenDnaCode

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST

- Reviewopmerking: Query verfijnen om false positives te voorkomen

  1. … 3 more files in changeset.
Is het niet veiliger om hier een ZoekViaToeleveringnSequentienummerEnRecent van te maken? sequentienummers van sommige ladetypes beginnen elke keer weer op 1 kwestie van geen false positives te h...

Is het niet veiliger om hier een ZoekViaToeleveringnSequentienummerEnRecent van te maken?

sequentienummers van sommige ladetypes beginnen elke keer weer op 1

kwestie van geen false positives te hebben

[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST
[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST
[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST

- Het afdrukken van klantetiketten SST zorgde voor een aanpassing van de sequetienummers in de dnacodetabel wat niet gewenst is. Dit zorgde voor een verdere verkeerde verwerking

  1. … 4 more files in changeset.
dit is idd één van de gevallekes waar opgelet gaat moeten worden

dit is idd één van de gevallekes waar opgelet gaat moeten worden

Kan het dat deze ook al aangepast was in ICT-2171? Geen idee of dit voor conflicten kan zorgen?

Kan het dat deze ook al aangepast was in ICT-2171? Geen idee of dit voor conflicten kan zorgen?

[ICT-2146] pa-ub-215: vhossetiketten volgorde dnacodes
[ICT-2146] pa-ub-215: vhossetiketten volgorde dnacodes
[ICT-2171] pa-ub-222: toelevering bijwerken: enkel recente records
[ICT-2171] pa-ub-222: toelevering bijwerken: enkel recente records
[ICT-2171] pa-ub-222: toelevering bijwerken: enkel recente records

- nieuwe method op repo gebruiken

  1. … 1 more file in changeset.
[ICT-2146] pa-ub-215: vhossetiketten volgorde dnacodes

- eerste versie, nog testen uitvoeren

  1. … 3 more files in changeset.
[ICT-1732] Implementatie SpaceStep Klantetiket

- Reviewopmerkingen

  1. … 5 more files in changeset.
Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True a...

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True als laatste parameter hebben.

{$$$False} ipv 0

{$$$False} ipv 0

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier...

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier houden we de interface(s) proper (zeker als er ooit nog andere parameters bij moeten komen).

Dit geldt ook voor sommige andere methods/klasses in deze review

comments weg

comments weg

[IC-1732] Implementatie SpaceStep Klantetiket
[IC-1732] Implementatie SpaceStep Klantetiket
[IC-1732] Implementatie SpaceStep Klantetiket

- Testen voorzien voor extra parameter IsKlantEtiketSpaceStep

Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequent...

Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequentienummer!?

juist een bedenking. Moet deze test ook voor klantetiketten?

juist een bedenking. Moet deze test ook voor klantetiketten?

[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken

- Bij herafdruk mag de dnacode bijgewerkt worden: afgezonderd en testjes toegevoegd

  1. … 1 more file in changeset.