Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Captcha #39

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Captcha #39

wants to merge 3 commits into from

Conversation

iromli
Copy link
Member

@iromli iromli commented Aug 24, 2014

This changeset introduces recaptcha in registration form.

screenshot from 2014-08-24 19 40 23

However, I'm not quite sure whether we need to show captcha in edit mode, so it's disabled in edit mode.

screenshot from 2014-08-24 19 40 37

@akbargumbira
Copy link
Member

sepertinya sih oke dari segi fungsionalitas. Tapi agak kurang sreg karna jadi besar sekali formnya.

Mas @za, apakah ada rencana lebih jauh ke depannya untuk project members ini? Saya baru port ini ke django (https://github.com/AIFDR/inasafe-user-map), yang menurut saya akan lebih mudah kl ingin diextend.

@iromli
Copy link
Member Author

iromli commented Sep 29, 2014

@akbargumbira apakah porting ke django menjawab issue #32 ? :)

@akbargumbira
Copy link
Member

Di portingan tersebut user harus konfirmasi registrasi (melalui email), sehingga walaupun ada yg buat bot script untuk registrasi, data yang ditambahkan tersebut tidak langsung ditampilkan.

@iromli
Copy link
Member Author

iromli commented Sep 30, 2014

@akbargumbira kita bisa porting fitur registrasi itu ke project ini?

@akbargumbira
Copy link
Member

@iromli tentu bisa sih

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants