nextcloud / tables

🍱 Nextcloud tables app
https://apps.nextcloud.com/apps/tables
GNU Affero General Public License v3.0
148 stars 23 forks source link

Applications: Enable multiple tables instances at the same time #807

Closed enjeck closed 8 months ago

enjeck commented 9 months ago

Is your feature request related to a problem? Please describe.

As part of https://github.com/nextcloud/tables/issues/311, we need to have multiple tables on same page.

Describe the solution you'd like

Refactor the global store to hold multiple tables by using the table ids as a key when storing rows and columns. Also, we need to ensure that modals are targeting the right table/view.

Describe alternatives you've considered

No response

Additional context

https://github.com/nextcloud/tables/issues/311#issuecomment-1790569092

enjeck commented 9 months ago

Lots of changes required here, so will split this across multiple PRs to make easier to review