automatisch / automatisch

The open source Zapier alternative. Build workflow automation without spending time and money.
https://automatisch.io
Other
4.2k stars 314 forks source link

Fix issue #1933 #1934

Closed jmas closed 1 week ago

jmas commented 2 weeks ago

Problem with rendering in DynamicField component with long variables #1933

This fix makes DynamicField look in next way:

Знімок екрана 2024-06-13 о 03 31 43
CLAassistant commented 2 weeks ago

CLA assistant check
All committers have signed the CLA.

jmas commented 2 weeks ago

@kattoczko Good catch. Sorry for another loop of review. I did fix (moved min-with: 0s to another DOM elements) and tested it locally:

Знімок екрана 2024-06-14 о 19 47 26 Знімок екрана 2024-06-14 о 19 48 05
barinali commented 2 weeks ago

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @jmas, I appreciate it. Thank you for your contribution to Automatisch! May I ask if you can proceed with the CLAassistant?

I quoted the reference bot-message from the CLAassistant above.

jmas commented 2 weeks ago

@barinali Thank you for point me on it. I signed it! Check: https://github.com/automatisch/automatisch/pull/1934#issuecomment-2164137434