Danny Martini

Results 96 comments of Danny Martini

> hey @despairblue thanks for adding the tests and cleaning up the implementation. :) I was going to look at the tests tomorrow! I appreciate it! > > Is there...

@guillaumejacquart @ivov @netroy I was wondering if this would make sense: https://github.com/n8n-io/n8n/compare/introduce-registered-entity-decorator-alternative-1 TLDR: 1. The `BaseN8nModule` enforces modules to implement an `entities` function that returns all entities 2. The module...

> Let's do the same for `workflow` package? @ivov done

@ivov could you check the LicenseState changes? Thanks ❤️

> Hey, code looks good to me. I don't know the execution logic enough, but I was just wondering if this did not break some hypothetical other cases when we...

@mutdmour Would this achieve the same? I tested it manually and it works in the cases I tried and is less intrusive: https://github.com/n8n-io/n8n/compare/sug-48-alternative-1?expand=1

@lukas-sieber can you share the workflow with us? And if possible a video showing exactly what you do? With that we should be able to find what's wrong. Right now...

@scropion86 The fix was only released in 1.99 yesterday, which is in beta right now. You could try that version or wait until it's stable next week. 🙏🏾

I pushed a commit updating the tests. The names were not describing the failure reasons and a test for this was missing: ``` typeof headers['x-forwarded-host'] === 'string' ? headers['x-forwarded-host'] :...

![image](https://github.com/user-attachments/assets/ba308e46-f983-4940-830c-2488e4e1ebec) Not sure what's going on here. Locally it works. May be a caching issue 🤔