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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

esa: Suppress article body from notificaton on post_update #52

Merged
merged 1 commit into from Nov 6, 2015

Conversation

Projects
None yet
2 participants
@kakutani
Contributor

kakutani commented Nov 5, 2015

I found I don't need the full-length article body in the notifications on post_update every time.
(a link to diff is enough for me).

So I'd like to suppress this 馃毧 馃毧 馃毧

Thanks in advance!

@hibariya

This comment has been minimized.

Show comment
Hide comment
@hibariya

hibariya Nov 6, 2015

Member

I found I don't need the full-length article body in the notifications on post_update every time.
(a link to diff is enough for me).

That makes sense. I think so, too 馃惓

Member

hibariya commented Nov 6, 2015

I found I don't need the full-length article body in the notifications on post_update every time.
(a link to diff is enough for me).

That makes sense. I think so, too 馃惓

hibariya added a commit that referenced this pull request Nov 6, 2015

Merge pull request #52 from kakutani/esa-suppress-body-on-update
esa: Suppress article body from notificaton on post_update

@hibariya hibariya merged commit 1534126 into idobata:master Nov 6, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@kakutani kakutani deleted the kakutani:esa-suppress-body-on-update branch Nov 6, 2015

@kakutani

This comment has been minimized.

Show comment
Hide comment
@kakutani

kakutani Nov 6, 2015

Contributor

馃毧 馃毧 馃毧

Contributor

kakutani commented Nov 6, 2015

馃毧 馃毧 馃毧

@hibariya

This comment has been minimized.

Show comment
Hide comment
@hibariya

hibariya Nov 6, 2015

Member

@kakutani I've deployed this change into production. Thank you 馃尰 馃毧 馃毧 馃毧

Member

hibariya commented Nov 6, 2015

@kakutani I've deployed this change into production. Thank you 馃尰 馃毧 馃毧 馃毧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment