Skip to content

Tap mambu sync - do not truncate bookmark date for filter#32

Open
gvillafanetapia wants to merge 22 commits into
singer-io:masterfrom
MuttData:master
Open

Tap mambu sync - do not truncate bookmark date for filter#32
gvillafanetapia wants to merge 22 commits into
singer-io:masterfrom
MuttData:master

Conversation

@gvillafanetapia

Copy link
Copy Markdown

Description of change

When calling Mambu API; send timezone aware datetime to filter creationDate AFTER

This way no matter what timezone is configured on the Mambu side, the API will interpret the filter correctly.

Manual QA steps

  • pylint
pylint tap_mambu -d missing-docstring -d logging-format-interpolation -d too-many-locals -d too-many-arguments
Your code has been rated at 9.42/10 (previous run: 9.42/10, +0.00)

Risks

Rollback steps

  • revert this branch

@cmerrick

Copy link
Copy Markdown
Contributor

Hi @gvillafanetapia, 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.

@cmerrick

Copy link
Copy Markdown
Contributor

You did it @gvillafanetapia!

Thank you for signing the Singer Contribution License Agreement.

@gvillafanetapia

Copy link
Copy Markdown
Author

Related issue #31

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