OpenEnergyPlatform / oeplatform

Repository for the code of the Open Energy Platform (OEP) website. The OEP provides an interface to the Open Energy Family
http://openenergyplatform.org/
GNU Affero General Public License v3.0
61 stars 19 forks source link

Data versioning is not visibly in place #456

Open Bachibouzouk opened 4 years ago

Bachibouzouk commented 4 years ago

There are many issues linked to this issue, I opened this one to have one place to look at

jh-RLI commented 2 years ago

@Ludee @wingechr Interesting issue, the linked issues also provide a good idea of what happened to the "edit table" function. My understanding is that editing is possible via the API and it was not entirely clear how to proceed with the UI to edit a table. So it was removed and suggested to implement it again later. What do you think about this?

wingechr commented 2 years ago

it is possible with the API, BUT...

there is a very big underlying problem:

updating rows only works if the table has a numerical id column as primary key. But this is not being enforced (you can create tables without that), see also #739

This should be addressed first before we enable anything on the UI, because it will just cause more confusion and problems.

wingechr commented 2 years ago

@jh-RLI actually, this issue is not closed by the mentioned pull request(the id column is just one part of it)

jh-RLI commented 2 years ago

@wingechr Yea that's true I just wanted to link it and GitHub assumes it will close :/ I will just update the first comment and link the PR there.