Skip to content

Conversation

@NMantek
Copy link

@NMantek NMantek commented Jul 11, 2025

No description provided.

@NMantek NMantek self-assigned this Jul 11, 2025
@NMantek NMantek changed the title Add ability to override email settings in condition blocks Add ability to override email settings with condition blocks Jul 11, 2025
@NMantek NMantek requested a review from tlueder July 11, 2025 10:10
@tlueder
Copy link

tlueder commented Jul 11, 2025

I'm not sure if I like this approach, the public properties in the models are readonly for a reason.
Ones something is parsed it's content and or state should be guaranteed for the code that follows.

And can you add a min example conf so I can see what you are trying to process?

@NMantek
Copy link
Author

NMantek commented Jul 11, 2025

Sure. Im trying to set a reciever address depending on the value of a select box. Looks something like this:

  1 {
    conditions.OR1.AND1 = 1.field=value
    isTrue {
      emailOverride {
        admin {
          toEmail = example@email.tld 
        } 
      }
    }
  }

Im not sure how else id implement that. I dont think the finisher config should handle conditions either, at least not as long as dedicated conditon blocks exist

@NMantek
Copy link
Author

NMantek commented Aug 20, 2025

Replaced with #146

@NMantek NMantek closed this Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants