search

raphaelm / python-sepaxml

SEPA Direct Debit XML generation in python
MIT License
110 stars 42 forks source link

Allow to Set End-To-End-Id #8

Closed mdornseif closed 6 years ago

mdornseif commented 6 years ago

This Patch allows to provide a user generated End-To-End ID. If not supplied, it is automatically generated, as it was before.

mdornseif commented 6 years ago

Nur mal so als Erklärung, was das soll: Auf eine Konto, auf dem auch sehr viele andere Transaktionen laufen werden Lastschriften eingezogen und die Kontoauszüge automatisiert ausgewertet. Da ist der End-To-End-Id eine nette Sache.

B2B ist in soweit nett, da die Bank nur 2 Arbeitstage lang zurück geben kann und der Debitor gar nicht. D.h. nach 3 Tagen hat man sein Geld sicher. Braucht aber für jede Kontoverbindung recht viel Kommunikation mit dem Schuldner und dessen Bank.

mdornseif commented 6 years ago

Ich versteh leider nicht ganz, was da bei Travis falsch läuft.

raphaelm commented 6 years ago

Nur mal so als Erklärung, was das soll: Auf eine Konto, auf dem auch sehr viele andere Transaktionen laufen werden Lastschriften eingezogen und die Kontoauszüge automatisiert ausgewertet. Da ist der End-To-End-Id eine nette Sache.

Jupp, gute Idee!

B2B ist in soweit nett, da die Bank nur 2 Arbeitstage lang zurück geben kann und der Debitor gar nicht. D.h. nach 3 Tagen hat man sein Geld sicher. Braucht aber für jede Kontoverbindung recht viel Kommunikation mit dem Schuldner und dessen Bank.

Ja, das würde ich mir nicht antun wollen, aber solange es so wenig Code ist, warum nicht supporten ;)

Ich versteh leider nicht ganz, was da bei Travis falsch läuft. Auch hier: ERROR: /home/travis/build/raphaelm/python-sepadd/tests/test_endtoendid.py Imports are incorrectly sorted. :)

isort -rc . fixt das automatisch.

codecov[bot] commented 6 years ago

Codecov Report

Merging #8 into master will increase coverage by 0.11%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #8      +/-   ##
=========================================
+ Coverage   96.68%   96.8%   +0.11%     
=========================================
  Files          10      11       +1     
  Lines         453     470      +17     
=========================================
+ Hits          438     455      +17     
  Misses         15      15
Impacted Files Coverage Δ
tests/test_endtoendid.py 100% <100%> (ø)
sepadd/debit.py 97.49% <100%> (+0.01%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3ad68f5...40f11fd. Read the comment docs.

raphaelm commented 6 years ago

Thanks! :)