Draft: Explore isolating authorization-related logic from IssuableFinder
What does this MR do and why?
- Create a new container class
Scopes::ProjectAuth - Move the
Projectscope-building methods off ofIssuableFinder::Paramsinto the container class - Restructure the methods a bit so we can clearly see dependencies and data flow
Next steps
- Validate specs passing/coverage
- Do something similar for
IssuableFinderitself now thatParamsis done - Try to identify global or "base" scopes that we would always want queries in this finder to resolve to
- Identify how the container class could be restructured to function more like a policy scope