-
Notifications
You must be signed in to change notification settings - Fork 8
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
How should we deal with project-specific stuff in Data Explorer? #6
Comments
@EvgeniiaVak this sounds like the right idea to me also: centralize code and make it reusable. I do also wish that there was some pluggable mechanism for overriding templates, extending behavior, etc similar to the how themes/plugins work in a CMS / DMS). For instance if someone wanted to:
I'm guessing that this is out of the scope of this project, but it something that I would like to continue to think about. |
@EvgeniiaVak I totally agree with you, however, there is some bespoke client requirements that should not be in master branch. |
@EvgeniiaVak I have a nice example right now:
There is way to fix it, e.g., we can have Problem is that over time we can end up with a lot of configurations.. |
Ok, but still interesting how we decide what is worth extracting to config and what is not. Maybe by counting feature requests by clients? Once a feature is requested more than once add it to master. But then we need to have some list of these features or something like that. If we don't do anything we might end up reinventing the wheel over and over. |
My assumption was that we want to make Data Explorer as reusable as possible (that's why there are no CSS for example - they will be defined in the themes), but also I thought that Data Explorer should be very centralized, meaning no project-specific branches and extending the common code when possible. But I see there is the EDS branch, so I might have been wrong.
My suggestion would be to never create project-specific branches if possible and add features that might be needed by other projects in the future to the common code.
@starsinmypockets @anuveyatsu what do you think about it?
The text was updated successfully, but these errors were encountered: