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.

Sign up for GitHub

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

Jump to bottom

Update subscription exports #378

Merged

pleek91 merged 1 commit into main from update-subscription-exports

Jan 19, 2024

Collaborator

pleek91 commented Jan 19, 2024

Description

The models for subscriptions were exported as lowercase which doesn't seem right for classes. So as a little cleanup I did the following

Renamed Manager to SubscriptionManager
Renamed Channel to SubscriptionChannel
Removed default exports for Subscription, SubscriptionChannel, and SubscriptionManager
Updated imports across the codebase


          Update exports

322eb10

pleek91 requested a review from a team as a code owner

January 19, 2024 19:40

znicholasbrown approved these changes

View reviewed changes

pleek91 merged commit 52bc06c into main

3 checks passed

pleek91 deleted the update-subscription-exports branch

January 19, 2024 19:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet