GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Captcha by iromli · Pull Request #39 · id-python/members

Conversation

@iromli

@iromli iromli commented Aug 24, 2014

Copy link
Copy Markdown
Member

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
iromli added 2 commits August 24, 2014 18:57
@iromli
add recaptcha in registration form
54ad20c
@iromli
show captcha only in registration form, but not in editing mode
d5bfcea
@iromli iromli mentioned this pull request Aug 24, 2014
Add captcha input on user_form #32
Open
@iromli
add recaptcha-client package to requirements.txt
104e6a7
@iromli
iromli force-pushed the captcha branch from d3b76d1 to 104e6a7 Compare August 28, 2014 19:20
@akbargumbira

akbargumbira commented Sep 29, 2014

Copy link
Copy Markdown
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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@iromli

iromli commented Sep 29, 2014

Copy link
Copy Markdown
Member Author

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@akbargumbira

akbargumbira commented Sep 30, 2014

Copy link
Copy Markdown
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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@iromli

iromli commented Sep 30, 2014

Copy link
Copy Markdown
Member Author

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@akbargumbira

akbargumbira commented Sep 30, 2014

Copy link
Copy Markdown
Member

@iromli tentu bisa sih
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@iromli @akbargumbira
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.