Skip to content

StufBG204 endpoint#404

Merged
mprins merged 42 commits into
masterfrom
feat/stuf-bg-endpoint
Feb 13, 2018
Merged

StufBG204 endpoint#404
mprins merged 42 commits into
masterfrom
feat/stuf-bg-endpoint

Conversation

@mprins

@mprins mprins commented Jan 23, 2018

Copy link
Copy Markdown
Member

In deze PR alleen verwerking van Mutatiesoort.T (toevoeging)

todo:

  • postgresql database migratie script
  • oracle database migratie script
  • mssql database migratie script
  • revert logfile aanpassingen (debug logging uitzetten)
  • integratietests fixen of verwijderen

@mprins mprins changed the title Feat/stuf bg endpoint StufBG204 endpoint Jan 23, 2018
@mprins mprins added this to the 1.6.0 milestone Jan 23, 2018
@mprins mprins modified the milestones: 1.6.0, 1.5.3 Feb 6, 2018
@mprins mprins requested a review from cvanlith February 6, 2018 17:20
@mprins

mprins commented Feb 6, 2018

Copy link
Copy Markdown
Member Author

Er zitten wat zaken in deze PR die ik nodig heb voor de t&t / gbav bericht verwerking, oa datamodel aanpassing.

Het zou mooi zijn als we een betere (integratie) test coverage hadden.

@mprins mprins requested a review from mtoonen February 6, 2018 17:27
b.setVolgordeNummer(index);
b.setObjectRef(getObjectRef(n));
b.setSoort(BrmoFramework.BR_BRP);
b.setDatum(new Date());

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moet dit niet getBestandsDatum() zijn ipv tijdstip van verwerken (new Date())? bestandsdatum wordt gevuld met kennisgeving.getStuurgegevens().getTijdstipBericht()

@mprins mprins merged commit 7d21970 into master Feb 13, 2018
@mprins mprins deleted the feat/stuf-bg-endpoint branch February 13, 2018 13:50
mprins added a commit that referenced this pull request Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants