Google recaptcha integration

[avatsaev]

Added Google recaptcha to discourage spammers from screwing with invite limits
Fixed deployment error on heroku (dist not found) by removed dist folder from gitignore

[jackmcdade]

Would love to see this merged.

[bmgdev]

+1

[jackmcdade]

I've been using this fork for a few weeks now but it has recently just broken. The ReCaptcha's fail every time. I wouldn't merge this without resolving the issue.

[jackmcdade]

@jaspergreenham If I knew i would fix it.

[jackmcdade]

@jaspergreenham No, @avatsaev did.

[jackmcdade]

@jaspergreenham It's a bit late to expect much response so quickly on a fork of a niche app. I recommend a little patience :)

[jackmcdade]

@jaspergreenham you're gonna recommend something else even after pinging @avatsaev in the middle of the night, and him fixing it within 8 hours?

I for one, am very grateful. Thanks @avatsaev!!

[avatsaev]

you're welcome

[julianschiavo]

I've asked @rauchg to merge, he asked for link, I gave, now waiting to hear back :).

[rauchg]

Thanks @jaspergreenham. The PR is not mergeable. I also don't have any screenshots or anything else to ascertain what this looks like, therefore making it difficult to merge :|

[jackmcdade]

We're using it on slack.statamic.com. You can check it out there.

On Oct 22, 2016, at 6:06 PM, Guillermo Rauch [email protected] wrote:

Thanks @jaspergreenham. The PR is not mergeable. I also don't have any screenshots or anything else to ascertain what this looks like, therefore making it difficult to merge :|

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

[passsy]

Thanks!

[rauchg]

@jackmcdade I'd like to only display it after pressing "get my invite". I'll adjust the PR myself

[lucymhdavies]

@rauchg any news on getting this PR updated and merged?

[chadwhitacre]

I also don't have any screenshots or anything else to ascertain what this looks like

---

---

---

[chadwhitacre]

The PR is not mergeable.

GitHub sez unknown repo. Is this PR orphaned? @avatsaev Can you make a new PR? Or update this one?

[rauchg]

@whit537

It has merge conflicts:

.gitignore
README.md
bin/slackin

[chadwhitacre]

I recommend closing in favor of #311.