-
Notifications
You must be signed in to change notification settings - Fork 2
WIP Add infogram processor #174
base: dev-stable
Are you sure you want to change the base?
Conversation
@DAreRodz is this still WIP? |
I move it back to Req. features until it's finished. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm only testing Frontibotito in review comments. @orballo, is it working?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing webhook.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing webhook.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approve with some comment.
component: 'iframe', | ||
props: { src: `https://e.infogram.com/${id}?src=embed`, title }, | ||
}), | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some random comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need some changes here!
Closes #165