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

feat: wise.com connector #23

Merged
merged 55 commits into from
Oct 6, 2022
Merged

feat: wise.com connector #23

merged 55 commits into from
Oct 6, 2022

Conversation

altitude
Copy link
Member

No description provided.

@altitude altitude marked this pull request as draft July 24, 2022 23:13
Antoine Gelloz and others added 3 commits July 25, 2022 10:38
@altitude altitude changed the title feat: add wise.com connector feat: wise.com connector Jul 26, 2022
gfyrag and others added 21 commits August 23, 2022 10:51
…ishkafka

chore: Expose kafka messages models and bump dependencies
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 1 to 2.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@v1...v2)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
…odeql-action-2

chore(deps): bump github/codeql-action from 1 to 2
Bumps [github.com/numary/go-libs](https://github.com/numary/go-libs) from 1.0.0 to 1.0.1.
- [Release notes](https://github.com/numary/go-libs/releases)
- [Commits](formancehq/go-libs@v1.0.0...v1.0.1)

---
updated-dependencies:
- dependency-name: github.com/numary/go-libs
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
…umary/go-libs-1.0.1

chore(deps): bump github.com/numary/go-libs from 1.0.0 to 1.0.1
@flemzord flemzord marked this pull request as ready for review September 5, 2022 10:00
Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 2 to 3.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v2...v3)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [docker/setup-qemu-action](https://github.com/docker/setup-qemu-action) from 1 to 2.
- [Release notes](https://github.com/docker/setup-qemu-action/releases)
- [Commits](docker/setup-qemu-action@v1...v2)

---
updated-dependencies:
- dependency-name: docker/setup-qemu-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
darkmatterpool and others added 7 commits September 26, 2022 14:33
Signed-off-by: Lawrence Zawila <[email protected]>
Signed-off-by: Lawrence Zawila <[email protected]>
…onnector-wise

# Conflicts:
#	cmd/server.go
#	pkg/bridge/integration/loader.go
Signed-off-by: Lawrence Zawila <[email protected]>
@codecov
Copy link

codecov bot commented Sep 28, 2022

Codecov Report

Merging #23 (89baaf9) into main (4948130) will increase coverage by 2.02%.
The diff coverage is 39.07%.

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   28.28%   30.31%   +2.02%     
==========================================
  Files          40       55      +15     
  Lines        2174     2896     +722     
==========================================
+ Hits          615      878     +263     
- Misses       1491     1932     +441     
- Partials       68       86      +18     
Impacted Files Coverage Δ
cmd/server.go 0.00% <ø> (ø)
pkg/api/http.go 0.00% <0.00%> (ø)
pkg/bridge/cdi/module.go 0.00% <0.00%> (ø)
pkg/bridge/connectors/wise/client.go 0.00% <0.00%> (ø)
pkg/bridge/connectors/wise/config.go 0.00% <0.00%> (ø)
pkg/bridge/connectors/wise/loader.go 0.00% <0.00%> (ø)
pkg/bridge/connectors/wise/task_fetch_profiles.go 0.00% <0.00%> (ø)
pkg/bridge/connectors/wise/task_fetch_transfers.go 0.00% <0.00%> (ø)
pkg/bridge/connectors/wise/task_resolve.go 0.00% <0.00%> (ø)
pkg/bridge/ingestion/ingester.go 55.55% <0.00%> (ø)
... and 47 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@flemzord flemzord requested review from a team and flemzord October 3, 2022 08:00
@flemzord flemzord self-assigned this Oct 3, 2022
Signed-off-by: Lawrence Zawila <[email protected]>
Signed-off-by: Lawrence Zawila <[email protected]>
viper.GetBool(authBearerUseScopesFlag),
dummypay.NewLoader(),
),
cdi.ConnectorModule(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@darkmatterpool I don't like this way of register connectors. It would be ideal if the connectors registers themselves to the core. If you have an idea for this....

return profiles, err
}

b, _ := io.ReadAll(res.Body)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe check the error?

return transfers, err
}

b, _ := io.ReadAll(res.Body)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe check the error?

gfyrag
gfyrag previously approved these changes Oct 5, 2022
antoinegelloz
antoinegelloz previously approved these changes Oct 5, 2022
Signed-off-by: Lawrence Zawila <[email protected]>
@darkmatterpool darkmatterpool merged commit 3e0498e into main Oct 6, 2022
@darkmatterpool darkmatterpool deleted the feat/connector-wise branch October 6, 2022 08:29
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.

6 participants