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

[1-2 days] [Master Issue] Implement Factory pattern (Factory Method or Abstract Factory) #9

Closed
will-r-wang opened this issue Mar 7, 2021 · 0 comments · Fixed by #25
Closed

[1-2 days] [Master Issue] Implement Factory pattern (Factory Method or Abstract Factory) #9

will-r-wang opened this issue Mar 7, 2021 · 0 comments · Fixed by #25
Assignees
@will-r-wang @alexatshopify
Projects
Course Management System

Comments

@will-r-wang
Copy link
Owner

will-r-wang commented Mar 7, 2021 •
edited

This is a significant part of our project (worth 15%) and thus should be composed carefully, thoroughly documented, and encompassed properly in one large PR. The use case of either of the two patterns will need to be properly justified and all team members should contribute to the implementation and code design of the work.

See #4
@will-r-wang will-r-wang created this issue from a note in Course Management System (To do) Mar 7, 2021
@will-r-wang will-r-wang changed the title Implement Factory pattern (Factory Method or Abstract Factory) [3-5 days] [Master Issue] Implement Factory pattern (Factory Method or Abstract Factory) Mar 7, 2021
@alexatshopify alexatshopify self-assigned this Mar 8, 2021
@will-r-wang will-r-wang moved this from To do to In progress in Course Management System Mar 12, 2021
@will-r-wang will-r-wang self-assigned this Mar 12, 2021
@will-r-wang will-r-wang changed the title [3-5 days] [Master Issue] Implement Factory pattern (Factory Method or Abstract Factory) [1-2 days] [Master Issue] Implement Factory pattern (Factory Method or Abstract Factory) Mar 12, 2021
@will-r-wang will-r-wang mentioned this issue Mar 13, 2021
Merged
2 tasks
Course Management System automation moved this from In progress to Done Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@will-r-wang will-r-wang

@alexatshopify alexatshopify

Labels
None yet
Projects
Course Management System
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Design Pattern] Add Factory Pattern will-r-wang/CMSLearn-Course-Management-System
2 participants
@will-r-wang @alexatshopify