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

Hotfix for PR#76 #79

Merged
merged 137 commits into from
Apr 21, 2022
Merged

Hotfix for PR#76 #79

merged 137 commits into from
Apr 21, 2022

Conversation

DownstreamDataTeam
Copy link
Contributor

Description of change

Fix for Audit Trail Processor crashing when bookmark is a list

Manual QA steps

  • Tested that the problem is actually solved, manually

Risks

Rollback steps

  • revert this branch

Radu Marinoiu and others added 30 commits February 14, 2022 09:05
Pull changes from Stitch

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!32
release/37 changes, squashed

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!37
…e/38'

[ECDDC-575] Refactor Clients stream

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!40
…ream'

# Conflicts:
#   tap_mambu/sync.py
#   tap_mambu/tap_mambu_refactor/main.py
…e/38'

[ECDDC-555] Refactor centres stream

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!39
…ream' into 'release/38'

[ECDDC-604] Added field "assigned_branch_key" to clients stream

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!42
… into 'release/38'

[ECDDC-607] Changed readme file to correct tap_config.json into config.json

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!44
…elease/38'

[ECDDC-601] Add code scanning with sonar

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!43
Radu Marinoiu and others added 23 commits March 28, 2022 17:02
…b.com:mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu into feature/ECDDC-657_Refactor-audit-trail-stream
…nto 'release/40'

[ECDDC-667] Added missing fields to loan_transaction's 'affected_amounts' object

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!81
…ease/40'

[ECDDC-658] Refactor activities stream

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!79
…l-stream'

# Conflicts:
#   refactor_tests/tap_processors/test_processors.py
#   tap_mambu/sync.py
#   tap_mambu/tap_mambu_refactor/helpers/generator_processor_pairs.py
#   tap_mambu/tap_mambu_refactor/tap_generators/generator.py
…lease/40'

[ECDDC-657] Refactor audit trail stream

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!80
Version 2.4.2 and changelog (singer-io#73)

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!82
…t' into 'release/40'

[ECDDC-674] Added old field back, so that the change is backwards compatible

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!84
… 'release/40'

[ECDDC-678] Fixed issue with generator values

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!92
# Conflicts:
#	tap_mambu/tap_mambu_refactor/tap_processors/processor.py
Merge master into release/40

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!99
[ECDDC-694] Hotfix for Audit Trail Processor Crash

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!104
@singer-bot
Copy link

Hi @DownstreamDataTeam, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@dmosorast dmosorast merged commit 7c0aea9 into singer-io:master Apr 21, 2022
@singer-bot
Copy link

You did it @DownstreamDataTeam!

Thank you for signing the Singer Contribution License Agreement.

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.

3 participants