Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Email property bcnMailTo could be refarctored to have a generic emailTo property #341

Open
rmathew1011 opened this issue Mar 5, 2024 · 1 comment
Open

Email property bcnMailTo could be refarctored to have a generic emailTo property #341

rmathew1011 opened this issue Mar 5, 2024 · 1 comment
Labels
tech debt When development teams take actions to expedite a deliverable.

Comments

@rmathew1011
Copy link
Contributor

No description provided.

@rmathew1011 rmathew1011 added the tech debt When development teams take actions to expedite a deliverable. label Mar 5, 2024
@kaladay
Copy link
Contributor

kaladay commented Mar 5, 2024

With the fw-cli, using generic names can be a problem when using the runAll/buildAll JS script.
There may be multiple workflows that all use different e-mails but use the sane address.

Having a separate name prevents this from the cli perspective and so if the change is to remove the specific e-mail variables to a generic emailTo, then the runAll/buildAll JS scripts cannot safely be used as-is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech debt When development teams take actions to expedite a deliverable.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rmathew1011 @kaladay