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

Continued refactor, added code scanning and added new fields #70

Merged
merged 39 commits into from
Mar 4, 2022

Conversation

DownstreamDataTeam
Copy link
Contributor

@DownstreamDataTeam DownstreamDataTeam commented Mar 2, 2022

Description of change

  • Refactored Centres, Clients and Branches streams
  • Added code scanning (Snyk and CodeClimate)
  • Added 'assignedBranchKey' field to Clients stream
  • Changed README.md to correct config.json name
  • Moved unit tests outside of the Tap Tester tests folder
  • Note: We will solve the requirements.txt issue in the next sprint, as this one is already over and we can't add more work to it

Manual QA steps

  • Ran unit tests, tested manually comparing before & after output of the Tap

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 7 commits March 1, 2022 10:27
…se/38'

[ECDDC-554] Refactor branches stream

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!45
[ECDDC-572] Added Snyk scanning

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

[ECDDC-625] Move unit tests (release 38)

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!47
@cmerrick
Copy link
Contributor

cmerrick commented Mar 2, 2022

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.

# Conflicts:
#	.gitlab-ci.yml
#	refactor_tests/tap_generators/test_generators.py
#	tap_mambu/sync.py
#	tap_mambu/tap_mambu_refactor/main.py
@DownstreamDataTeam DownstreamDataTeam changed the title Release/38 Continued refactor, added code scanning and added new fields Mar 2, 2022
Misc bug fixes and features, continued the refactor (singer-io#68)

See merge request mambucom/product/ecosystem/mambu-marketplace/connectors/singer/tap-mambu!49
@DownstreamDataTeam DownstreamDataTeam marked this pull request as ready for review March 2, 2022 09:19
@dmosorast dmosorast merged commit 922248a into singer-io:master Mar 4, 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.

4 participants