villebro commented on PR #31934:
URL: https://github.com/apache/superset/pull/31934#issuecomment-3215589760
> love the idea of general organization this way
>
> ```
> superset/
> {feature_or_subject_area}/models.py
> {feature_or_subject_area}/daos.py
> {feature_or_subject_area}/api.py
> {feature_or_subject_area}/views.py
> ```
>
> Super great but super disruptive and merge conflicting. Some things are
kind of going that way area, you could go one-per-PR.
>
> We'd need to identify the set of feature_or_subject_area, some might be
pretty tangled up with one-another ... In practice might be hard to segment
Totally, I've also been pushing for reorganizing the backend like this.
Commands should also go directly under the main feature/subject area. I believe
we actually already had commands and DAOs under the main feature before, but it
was moved in under a centralized package at some point. I propose doing this
massive refactor next time the breaking window opens to clean up the backend,
as it will make things much more modular.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
