Skip to content

Check if is array on SuppressionResource#40

Closed
mhortulanus wants to merge 4 commits intomainfrom
fix/key-to
Closed

Check if is array on SuppressionResource#40
mhortulanus wants to merge 4 commits intomainfrom
fix/key-to

Conversation

@mhortulanus
Copy link
Copy Markdown
Contributor

@mhortulanus mhortulanus commented Apr 2, 2025

This pull request includes a change to the table method in the SuppressionResource class to handle cases where the to field is a JSON-encoded string.

@mhortulanus mhortulanus requested a review from markvaneijk April 2, 2025 12:44
@arshaviras
Copy link
Copy Markdown

arshaviras commented Apr 7, 2025

Hi, tried your code, getting error Unknown named parameter $stream_id.
stream_id is NULL everywhere in mails table.

@Baspa
Copy link
Copy Markdown
Contributor

Baspa commented May 19, 2025

Is this still relevant or can it be closed? @arduinomaster22

@markvaneijk
Copy link
Copy Markdown
Contributor

Moet dit niet $record→mail→to zijn?

@Baspa
Copy link
Copy Markdown
Contributor

Baspa commented May 19, 2025

Moet dit niet $record→mail→to zijn?

Als dat zo is dan heb ik m al gefixt @markvaneijk

@markvaneijk
Copy link
Copy Markdown
Contributor

Voor de zekerheid moet het even getest worden :-)

@arshaviras
Copy link
Copy Markdown

Hi guys, why you ignoring my comment? :)))

@markvaneijk
Copy link
Copy Markdown
Contributor

Hi guys, why you ignoring my comment? :)))

Just released v1.4.3 that should solve your issue :)

@arshaviras
Copy link
Copy Markdown

Excellent :)

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.

4 participants