YunoHost-Apps / n8n_ynh

n8n package for YunoHost
https://n8n.io/
GNU General Public License v3.0
14 stars 19 forks source link

add email var #23

Closed ericgaspar closed 2 years ago

ericgaspar commented 2 years ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

ericgaspar commented 2 years ago

!testme

yunohost-bot commented 2 years ago

May the CI gods be with you! Test Badge

ericgaspar commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:rocket: Test Badge

ericgaspar commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Meow :cat2: Test Badge

fflorent commented 2 years ago

LGTM, thanks!

One question though: the email field seems required. I guess the user has to manually fill the new field if she/he upgrades the application.

Is there any mechanism to ask the value of this field? Or does he/she have to be vigilant about what the upgrade implies about the fields?

Thanks in advance!

ericgaspar commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:sunflower: Test Badge