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

chore: change ConnectionRepository to a hook #5629

Merged
merged 39 commits into from
Apr 4, 2024
Merged

Conversation

kmruiz
Copy link
Contributor

@kmruiz kmruiz commented Mar 27, 2024

Description

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@kmruiz kmruiz self-assigned this Mar 27, 2024
@kmruiz kmruiz force-pushed the feature/COMPASS-7775 branch 3 times, most recently from e0d6df5 to 3739c26 Compare April 2, 2024 14:55
@kmruiz kmruiz marked this pull request as ready for review April 2, 2024 15:14
packages/compass-connections/src/connections-manager.ts Outdated Show resolved Hide resolved
packages/compass-connections/src/provider.ts Outdated Show resolved Hide resolved
@kmruiz kmruiz merged commit 0f38ad5 into main Apr 4, 2024
12 of 13 checks passed
@kmruiz kmruiz deleted the feature/COMPASS-7775 branch April 4, 2024 14:57
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