Skip to content
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

Suggestion about tables. #958

Closed
jalonsob opened this issue Aug 16, 2016 · 3 comments
Closed

Suggestion about tables. #958

jalonsob opened this issue Aug 16, 2016 · 3 comments

Comments

@jalonsob
Copy link

I have seen that the name of each table for our dashboard is called with the same name like the original db. For example, if I have a table of a database called "source" and I want to use this table in Caravel I have to call it with the same name. It is saved in my caravel.db situated in the path indicated in caravel_config.py. But we can have a problem that I am having now. If I have two differents databases that I wanna use in caravel, and this databases use a table with the same name, I cant have the two tables of differents databases in two differents tables in Caravel. I think that it could be a good change to add a new column in "Tables" section called "name" to have two differents tables from differents databases but with the same name in each original database.

Now if we try to do this we get the next error in Caravel:

Integrity error, probably unique constraint

error

@xrmx
Copy link
Contributor

xrmx commented Aug 16, 2016

That's what verbose_name is for in the table model. Anyway regarding the integrity error it should already be fixed in master. Which version are you running? Which db are you using for caravel data?

@jalonsob
Copy link
Author

jalonsob commented Aug 17, 2016

I am using Caravel 0.10.0 and the database in the caravel_config.py is:

SQLALCHEMY_DATABASE_URI = 'sqlite:///.../caravel.db'

@xrmx
Copy link
Contributor

xrmx commented Aug 17, 2016

Closing then, feel free to reopen if doesn't work in the next release.

@xrmx xrmx closed this as completed Aug 17, 2016
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
* feat(plugin-chart-table): subject Add tooltip show multiple sort

* Change message
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
* feat(plugin-chart-table): subject Add tooltip show multiple sort

* Change message
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
* feat(plugin-chart-table): subject Add tooltip show multiple sort

* Change message
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
* feat(plugin-chart-table): subject Add tooltip show multiple sort

* Change message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants