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

Database access refactoring [WIP] #67

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Database access refactoring [WIP] #67

wants to merge 4 commits into from

Conversation

RheaAyase
Copy link
Member

Open for feedback.

@RheaAyase RheaAyase self-assigned this Apr 21, 2019
@RheaAyase
Copy link
Member Author

Added only very basic class to begin with... To be improved as I hit some specific use-cases that may require different approach.

@RheaAyase RheaAyase force-pushed the db-access-manager branch 2 times, most recently from aa017da to 530fd4f Compare April 21, 2019 23:33
@RheaAyase
Copy link
Member Author

So I've implemented one use-case of updating usernames/nicknames so far. We may not actually need half the generic methods in the end... we'll see what I hit when I dive into rewriting all the features that access user data.

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

@RheaAyase RheaAyase

Labels
1 - High Priority enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RheaAyase