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.

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

Custom property for partial message text tokens. #24

Open
wants to merge 1 commit into
base: 7.x-1.x
Choose a base branch
from

Conversation

jgullstr
Copy link

Currently, AFAIK, there's no way to get a token for a single delta value for a message text. I have a message type, where text (delta 0) holds an SMS text and (delta 1) holds an email texts. Both SMS and e-mails are sent by Rules, and I need to retrieve the delta values as tokens to their respective rule.
I've added a property, that retrieves the texts as a list, so tokens can use the delta values.

In order for this to work, there's an issue with chained tokens and Entity tokens that needs to be resolved. (See here)

@amitaibu
Copy link
Member

@jgullstr Thanks, any chance for adding a test?

@RoySegall please review.

@jgullstr
Copy link
Author

jgullstr commented May 6, 2015

@amitaibu As this approach is dependent on the chained token issue being resolved in Entity token, I'll wait until that is fixed prior to spending any time on writing tests.

@jgullstr jgullstr closed this May 6, 2015
@jgullstr
Copy link
Author

jgullstr commented May 6, 2015

Oops, accidental close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants