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

feature/smart connected account tracking #14

Merged
merged 16 commits into from
Jun 13, 2022

Conversation

gfyrag
Copy link
Contributor

@gfyrag gfyrag commented Jun 7, 2022

  • Fix panic on stripe pool.
  • Test stripe worker pool.
  • Be smart on when fetching conencted account transactions.
  • Fix bad account.
  • Add index on payment provider.
  • Fix concurrent read/write on map.
  • Add index on type.
  • Add index on payment references.
  • Add handling of payout_cancel and adjustment objects.
  • Catch panic and display a log.
  • Add a dedicated logger for potential otlp errors.
  • Skip stripe transactions if source is not defined.
  • Fix logging.
  • DEBUG.
  • Send payments events outside mongo transactions.
  • Remove MongoDB monitor.
  • Refine stripe balance transaction mapping.

@codecov
Copy link

codecov bot commented Jun 7, 2022

Codecov Report

Merging #14 (e6942cd) into main (48f50ba) will increase coverage by 2.73%.
The diff coverage is 56.01%.

❗ Current head e6942cd differs from pull request most recent head 6c87e44. Consider uploading reports for the commit 6c87e44 to get more accurate results

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   22.67%   25.40%   +2.73%     
==========================================
  Files          31       32       +1     
  Lines        1645     1712      +67     
==========================================
+ Hits          373      435      +62     
- Misses       1253     1258       +5     
  Partials       19       19              
Impacted Files Coverage Δ
cmd/root.go 18.78% <0.00%> (+0.10%) ⬆️
pkg/bridge/ingester.go 4.05% <0.00%> (-0.03%) ⬇️
pkg/bridge/manager.go 0.00% <0.00%> (ø)
pkg/database/indexes.go 0.00% <ø> (ø)
pkg/payment.go 26.00% <ø> (ø)
pkg/bridge/connectors/stripe/translate.go 20.80% <6.34%> (-8.05%) ⬇️
pkg/bridge/connectors/stripe/pool.go 71.64% <66.66%> (+71.64%) ⬆️
pkg/bridge/connectors/stripe/runner.go 79.54% <80.00%> (+6.54%) ⬆️
pkg/bridge/connectors/stripe/scheduler.go 81.61% <89.13%> (+3.66%) ⬆️
pkg/bridge/connectors/stripe/timeline_trigger.go 90.90% <90.90%> (ø)
... and 6 more

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 afe7c58...6c87e44. Read the comment docs.

@gfyrag gfyrag force-pushed the feature/smart-connected-account-tracking branch from e6942cd to 6c87e44 Compare June 13, 2022 09:48
@gfyrag gfyrag merged commit cc1ae8d into main Jun 13, 2022
@gfyrag gfyrag deleted the feature/smart-connected-account-tracking branch June 13, 2022 09:53
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