Skip to content

Refactoring & redesigning for stable version #232

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

Open
1 of 5 tasks
noplanman opened this issue Jun 26, 2016 · 3 comments
Open
1 of 5 tasks

Refactoring & redesigning for stable version #232

noplanman opened this issue Jun 26, 2016 · 3 comments

Comments

@noplanman
Copy link
Member

noplanman commented Jun 26, 2016

This issue is intended to be a todo list of the main code refactoring and redesigning that should preferably be completed before a stable version gets released.

Each item on the list has it's own issue (which is linked), which is the main point of conversation.
Conversation here on this issue should be related to the general idea of code refactor/redesign and which vital components are in need of it.

As always, open to discussion, suggestions, etc.

Refactor

Redesign

  • Request & Update procedure (Decouple the request and update functionality more to make it more logical and especially testable)
  • Conversations (Make the creation of conversations more streamlined and easier for the users)
  • Database handlers (optional?) (More abstract database handling to allow different database types by simply implementing the various handlers, e.g. for PostgreSQL, Redis, MongoDB, etc.)

@akalongman @MBoretto @jacklul
Please have a think about these and give your feedback so that we can get this show on the road 😉

@salemgolemugoo
Copy link

Logging. We shouldn't throw TelegramExceptions when monolog instance is provided for the API.

@MBoretto
Copy link
Collaborator

MBoretto commented Aug 29, 2016

@noplanman Can we close this
#106
since this list include the commander class?

@noplanman
Copy link
Member Author

Absolutely, have closed it 👍

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

No branches or pull requests

4 participants