nextcloud / tables

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

feat: Add events for row added and row updated #1101

Open come-nc opened 1 month ago

enjeck commented 1 month ago

Ah, I guess related to Workflows?

come-nc commented 1 month ago

Ah, I guess related to Workflows?

Yes!

juliushaertl commented 3 weeks ago

@come-nc To evaluate what we want to pass along in the event, could you share more insights on the use case you're covering with your workflow task?

come-nc commented 3 weeks ago

@come-nc To evaluate what we want to pass along in the event, could you share more insights on the use case you're covering with your workflow task?

Usecases are multiple:

I feel like putting the whole row content information in the event is too much because it may be a lot of data, while putting only the id allows to fetch the data from the API if needed. Not entirely sure. Maybe content but capped at a given maximum length would make sense?

The event will need to be serialized by implementing upcomming OCP\EventDispatcher\IWebhookCompatibleEvent interface.