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

initial OOD implementation #162

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

Rupeshiya
Copy link
Member


name: Pull Request template
about: Describe the PR.
title: ''
labels: ''
assignees: ''


Problem

Github Issue Number: #159
As of now, the codebase does not follow any design pattern, so it's pretty hard to make it maintainable and flexiblel_

Solution of problem

Refactor the backend codebase by following the Object-oriented design pattern like Singleton classes, factory classes

Type of Change

[ ] Bug fix
[✓] New Feature
[ ] Development of UI/UX prototypes
[ ] Small refactor
[ ] Change in Documentation

Checklist

[✓] My code follows the same style as the codebase
[ ] My Code change requires a change in documentation
[ ] I have updated the Readme accordingly
[✓] I made PR against development branch
[ ] I have run the test cases locally and it's passing.
[✓] I have squashed my commits

@Rupeshiya Rupeshiya force-pushed the design_pattern branch 2 times, most recently from 6f6cc0d to 395bef4 Compare July 30, 2020 07:00
@devesh-verma
Copy link
Member

Best way would have been going one file at a time of functionality wise. 😔

@Rupeshiya
Copy link
Member Author

Rupeshiya commented Jul 30, 2020

@devesh-verma Is there anything wrong?
Just fetch this PR locally and run. It is working fine.
Let me know if I need to change anything.

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

Successfully merging this pull request may close these issues.

3 participants