Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support v2.01 GRRCN files with v1.01 back compat #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leejo
Copy link

@leejo leejo commented Jul 21, 2021

v2.01 was release in October 2019, v3.01 was release in April 2021 but
no support for v3.01 is added here (yet)

from the Amex "Raw Data - Global Reconciliation (GRRCN) Technical File
Specification April 2021":

"v1.01 of the GRRCN file format will be decommissioned as of October 31,
2021, any files sent as v1.01 will be migrated to v2.01 of the GRRCN
file format. All customers using v1.01 of the file format will be
automatically moved to v2.01 on October 31, 2021." so there is some
urgency here as it is likely (at least from our testing) that v2.01 will
break with the current implementation.

the additional field here is FILLER in the TRANSACTION, SUBMISSION, and
TAXRECORD lines. the code has been tweaked to use the VERSION value in
the header and dynamically set the field_map depending on that, so it
retains backwards compatibility. a test and example file has been added
for v2.01 of the GRRCN file

@leejo leejo force-pushed the humanstate/GRRCN_v2.01_support branch from ce0f6a6 to 807bf73 Compare July 21, 2021 09:59
v2.01 was release in October 2019, v3.01 was release in April 2021 but
no support for v3.01 is added here (yet)

from the Amex "Raw Data - Global Reconciliation (GRRCN) Technical File
Specification April 2021":

"v1.01 of the GRRCN file format will be decommissioned as of October 31,
2021, any files sent as v1.01 will be migrated to v2.01 of the GRRCN
file format. All customers using v1.01 of the file format will be
automatically moved to v2.01 on October 31, 2021." so there is some
urgency here as it is likely (at least from our testing) that v2.01 will
break with the current implementation.

the additional field here is FILLER in the TRANSACTION, SUBMISSION, and
TAXRECORD lines. the code has been tweaked to use the VERSION value in
the header and dynamically set the field_map depending on that, so it
retains backwards compatibility. a test and example file has been added
for v2.01 of the GRRCN file
@leejo leejo force-pushed the humanstate/GRRCN_v2.01_support branch from 807bf73 to 5dc3c11 Compare July 21, 2021 09:59
@tomheady
Copy link
Contributor

Looks good, thanks for the patch!

tomheady pushed a commit to tomheady/Finance-AMEX-Transaction that referenced this pull request Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants