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.

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

Factory Class for Calendar Connectors #162

Open
MelissaAutumn opened this issue Oct 23, 2023 · 1 comment
Open

Factory Class for Calendar Connectors #162

MelissaAutumn opened this issue Oct 23, 2023 · 1 comment
Labels
enhancement New feature or request maintenance A task or required work to maintain the application
Milestone
1.0 Open

Comments

@MelissaAutumn
Copy link
Member

Right now we manually do a if/else to create the approriate calendar connector. Since these should be within the same interface (same function calls)

Instead we could just have a function/class that creates the approriate calendar connector and that would clean things up a bunch.
@MelissaAutumn MelissaAutumn added the enhancement New feature or request label Oct 23, 2023
@devmount devmount added the maintenance A task or required work to maintain the application label Oct 24, 2023
@devmount devmount added this to the v0.2 milestone Oct 24, 2023
@MelissaAutumn MelissaAutumn modified the milestones: v0.2, 1.0 Open Feb 28, 2024
@MelissaAutumn
Copy link
Member Author

We need to adjust the constructor for the connector classes to match. I'm not sure the best approach here but it's low priority so moving this to 1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request maintenance A task or required work to maintain the application
Projects
None yet
Milestone
1.0 Open
Development

No branches or pull requests
2 participants
@devmount @MelissaAutumn