Comments (14)
I've sent an email to the AG Grid team just to confirm that there won't be any licensing/EULA issues to contend with in our project, which might be used for commercial or non-commercial purposes. This seems like the occasion to ask for permission rather than forgiveness ;)
Also, the SIP makes no mention of converting the Table viz plugin to AG Grid, but assume that's a logical next step in the table roadmap if this phase goes according to plan?
from superset.
the SIP makes no mention of converting the Table viz plugin to AG Grid, but assume that's a logical next step in the table roadmap if this phase goes according to plan?
That's right.
from superset.
I've sent an email to the AG Grid team just to confirm that there won't be any licensing/EULA issues to contend with in our project, which might be used for commercial or non-commercial purposes. This seems like the occasion to ask for permission rather than forgiveness ;)
according to repo of ag-grid has a MIT license https://github.com/ag-grid/ag-grid?tab=MIT-1-ov-file#readme
from superset.
cc: #24318
from superset.
cc: #24319
from superset.
@justinpark also consider in the DataTable in chart-plugins the option to custom render of column for example we want present a list of tweets in a table with a column that renders to a hyperlink to navigate to a original tweet in twitter site.
I would like to contribute to the change of DataTable in chart-plugins
from superset.
we want present a list of tweets in a table with a column that renders to a hyperlink to navigate to a original tweet in twitter site.
I would like to contribute to the change of DataTable in chart-plugins
Your contribution to this change would be highly appreciated, and it can be pursued independently of the dependencies mentioned in the proposal. This is because this proposal involves overriding the existing column rendering logic, allowing you to implement the desired hyperlink functionality separately.
from superset.
I've sent an email to the AG Grid team just to confirm that there won't be any licensing/EULA issues to contend with in our project, which might be used for commercial or non-commercial purposes. This seems like the occasion to ask for permission rather than forgiveness ;)
Much appreciated @rusackas ❤️
Also, the SIP makes no mention of converting the Table viz plugin to AG Grid, but assume that's a logical next step in the table roadmap if this phase goes according to plan?
@justinpark I think it's important to add this to the SIP so folks that are not participating in the discussion can understand what's the plan. This depends on the reply of the Ag Grid folks and I can see two scenarios:
- Orgs are not allowed to use the community version of the table when commercializing Superset. In this scenario, all tables still default to AgGrid but we need to add a configuration to disable it and show the old table instead.
- No license restriction. We replace all tables and remove the old table.
During the SIP Office Hours, @yousoph also asked if there is any feature present in the current table that would not be available in AG Grid.
All this content could go under the Migration Plan and Compatibility section. We also need to add an example of how our theme could be applicable to the table to ensure visual consistency.
from superset.
Your contribution to this change would be highly appreciated
It would be a pleasure to contribute, could you give me a guide on the structure of the project to familiarize myself with the repository, if that is possible,
Is this the main thing to change?
https://github.com/apache/superset/tree/master/superset-frontend/plugins/plugin-chart-pivot-table
from superset.
I think it's important to add this to the SIP so folks that are not participating in the discussion can understand what's the plan. This depends on the reply of the Ag Grid folks and I can see two scenarios:
- Orgs are not allowed to use the community version of the table when commercializing Superset. In this scenario, all tables still default to AgGrid but we need to add a configuration to disable it and show the old table instead.
- No license restriction. We replace all tables and remove the old table.
We've also received confirmation from the ag-Grid team regarding the use of the community version in Superset. In light of this, this proposal suggests replacing all tables with ag-Grid.
from superset.
We've also received confirmation from the ag-Grid team regarding the use of the community version in Superset. In light of this, this proposal suggests replacing all tables with ag-Grid.
Wooohoooo! 🎉
from superset.
We've also received confirmation from the ag-Grid team regarding the use of the community version in Superset.
Indeed. It looks like I forgot to mention that here, so thank you. Specifically, they stated in an email:
AG Grid community is free to download and use without the need to notify us.
As long as you are not using any of the features of AG Grid enterprise these will be no issue.
from superset.
Will #26364 column filter like the one in a be implemented?
from superset.
Will #26364 column filter like the one in a be implemented?
It can be a good idea for a follow-up improvement, but it is not included in the initial implementation. The implementation of a column filter like the one in #26364 would require further consideration and planning.
from superset.
Related Issues (20)
- [SIP-134] Hide inactive users HOT 3
- Unable to retrieve Datasets from SST Api HOT 8
- Alerts & Reports - CSV as Message Content resulting HTTP 500 HOT 4
- Can't access database in SQL lab HOT 6
- When requesting a Guest Token API, error code 422 is reported HOT 1
- Fatal error while saving query as dataset with Doris and {{ from_dttm }} & {{ to_dttm }} HOT 2
- Query saved paramters HOT 4
- "View as table" option on graphs/slices does not bypass the cache HOT 1
- [SIP-135] Proposal for aggregate dimensions HOT 3
- Error in Add Alert form when upgrading to version 4.0.1 HOT 4
- how to edit dd.mm.yyyy dateformat in apache superset-ui-chart-controls HOT 1
- Pivot Table (legacy) sum issues HOT 2
- Login shows endless loading HOT 1
- Superset 4.0.1 uses again outdated setuptools 58.1.0 with CVE HOT 2
- missing redis-update in superset 4.0.1 => redis 4.6.0 HOT 2
- superset 4.0.1 uses Gunicorn 21.2.0 with CVE => should update to 22.0.0 HOT 6
- superset 4.0.1 uses werkzeug 3.0.1 with CVE => should update to 3.0.3 or newer HOT 2
- superset 4.0.1 uses sqlparse 0.4.4 with two high CVE findings => update to 0.5.0 HOT 2
- Doris made an error in creating a chart using the GROUP_CONCAT syntax HOT 2
- Content not from webpack is served from 'D:\tech_release\superset-fork\superset\static\assets' directory HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from superset.