Nicola Puppa

Results 34 comments of Nicola Puppa

> My idea is to add another field besides `id` in episode.json, `ids` that would be an array of strings: > > ```js > { > "id": "uZZoDaVdCAE", > "ids":...

> But if we are to remove `id`, probably we should first deprecate that field, give a warning in the logs, and after several releases remove the field completely. I...

> When GitHub action fails, I receive an email from GitHub that the action failed, and this is very useful, because in most cases from my experience, the action fails...

> Nope, this is about competing a job. The one you linked seems to be related to process instance creation. I suppose you could consider it to be related as...

> Thanks Nic. Personally I'd prefer to not merge this yet as this will conflict with the changes for multi-tenancy. I'm agree with you, if you see conflict with multitenancy...

Hey @remcowesterhoud this PR is still valid for you ? Wdyt ?

I rebased the branch to the current main, and added multi tenancy and PIT support. I still need to run a benchmark to understand the impact of the fk addition

> @nicpuppa this needs backporting down to 8.1, right? I think so

Thanks for the review @koevskinikola, I applied all the suggestions. Please have another look 👀