Skip to content

Conversation

@neuhoffm
Copy link
Contributor

As we discussed in #1. Sorry for the delay :(

Copy link
Owner

@dhimmel dhimmel left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

I hadn't used IFTTT before, but in theory it seems like a more versatile and flexible notification solution compared to the Telegram bot. The interface was a bit unintuitive for me, but I was able to follow the instructions all the way through. At the end I created an applet described as:

If Maker Event "hackjohn", then Send a rich notification from the IFTTT app Activity

The part that took me the longest was actually realizing that I had to click "+" sign to begin creating the applet.

image

I made a few small suggestions, if you want to accept those, then I will merge this PR.

# Set enable_ifttt to True to receive IFTTT notification
enable_ifttt = True
event_name = 'hackjohn'
ifttt_key = 'cJLFuy0CHPBwK0wqndI2g9'
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
ifttt_key = 'cJLFuy0CHPBwK0wqndI2g9'
ifttt_key = 'replace-with-ifttt-key'

Copy link
Owner

Choose a reason for hiding this comment

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

@neuhoffm, do you not think this is a worthwhile change? Is cJL.... a real key or just an example? We will want to do whatever is most clear to let users know that they must replace the value with their key.

hackjohn.py Outdated
## Notifications using IFTTT

# Set enable_ifttt to True to receive IFTTT notification
enable_ifttt = True
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
enable_ifttt = True
enable_ifttt = False

let's default to false since users will have to populate the ifttt fields first

ifttt_key = 'cJLFuy0CHPBwK0wqndI2g9'

# Create you own IFTTT (https://ifttt.com/create)
# Select webhooks for this and select receive a web trigger
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
# Select webhooks for this and select receive a web trigger
# Select webhooks for this and select "Receive a web request".

for referencem, here are some URLs that the process took me to:

https://ifttt.com/create/connect-maker_webhooks?sid=3
https://ifttt.com/create/if-receive-a-web-request?sid=9
https://ifttt.com/create/if-receive-a-web-request-then-send-a-rich-notification-from-the-ifttt-app?sid=14

Co-Authored-By: Daniel Himmelstein <daniel.himmelstein@gmail.com>
@dhimmel dhimmel merged commit 0e8c69e into dhimmel:master Nov 18, 2019
dhimmel added a commit that referenced this pull request Nov 18, 2019
Follows up on #4
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.

2 participants