Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

WIP Add infogram processor #174

Open
wants to merge 1 commit into
base: dev-stable
Choose a base branch
from
Open

Conversation

DAreRodz
Copy link
Contributor

@DAreRodz DAreRodz commented Oct 29, 2018

Closes #165

@DAreRodz DAreRodz self-assigned this Oct 29, 2018
@DAreRodz DAreRodz requested a review from luisherranz October 29, 2018 10:46
@luisherranz luisherranz removed their request for review October 29, 2018 10:49
@luisherranz
Copy link
Contributor

@DAreRodz is this still WIP?

@luisherranz
Copy link
Contributor

I move it back to Req. features until it's finished.

Copy link
Contributor

@luisherranz luisherranz left a 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?

@luisherranz luisherranz requested a review from orballo December 31, 2018 09:32
Copy link

@orballo orballo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing webhook.

Copy link

@orballo orballo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing webhook.

Copy link

@orballo orballo left a 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 },
}),
};
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some random comment.

Copy link

@orballo orballo left a 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!

orballo
orballo previously approved these changes Dec 31, 2018
@orballo orballo dismissed their stale review December 31, 2018 13:02

It was a test

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

Successfully merging this pull request may close these issues.

3 participants