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

Properly seperate out BanchoUser and User classes #200

Open
cg0 opened this issue Feb 28, 2022 · 0 comments
Open

Properly seperate out BanchoUser and User classes #200

cg0 opened this issue Feb 28, 2022 · 0 comments
Labels
bancho Issues related to Bancho bug Something isn't working enhancement New feature or request

Comments

@cg0
Copy link
Member

cg0 commented Feb 28, 2022

Right now a User class does exist but we rely on BanchoUser a lot and we should:

  • Only use BanchoUser when it's a task that requires access to a User on Bancho
  • Move everything that isn't relating to a user on Bancho to User
  • Make it easier to get User objects
@cg0 cg0 added bug Something isn't working enhancement New feature or request bancho Issues related to Bancho labels Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bancho Issues related to Bancho bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant