Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Refactor the whole context mess as its own managed class/type #147

Open
JaneJeon opened this issue Dec 27, 2020 · 1 comment
Open

Refactor the whole context mess as its own managed class/type #147

JaneJeon opened this issue Dec 27, 2020 · 1 comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@JaneJeon
Copy link
Owner

JaneJeon commented Dec 27, 2020

...since the context about the context is NOT passed in-between methods currently (so you have no idea what fields ARE available on the context, and what type they are!)

Right now everything's held together with fucking duct tape of random functions EVERYWHERE

@JaneJeon
Copy link
Owner Author

JaneJeon commented Dec 27, 2020

Wanna take this one on? @sssss465 You'd need to merge in the changes to the next branch: https://github.com/JaneJeon/objection-authorize/tree/next

@JaneJeon JaneJeon added good first issue Good for newcomers help wanted Extra attention is needed labels Dec 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant