Skip to content

Add captcha support to OpenID based signup - #1307

Merged
bkcsoft merged 1 commit into
go-gitea:masterfrom
strk:openid-captcha
Mar 20, 2017
Merged

Add captcha support to OpenID based signup#1307
bkcsoft merged 1 commit into
go-gitea:masterfrom
strk:openid-captcha

Conversation

@strk

@strk strk commented Mar 17, 2017

Copy link
Copy Markdown
Member

No description provided.

@lunny lunny added this to the 1.2.0 milestone Mar 18, 2017
@lunny lunny added the type/enhancement An improvement of existing functionality label Mar 18, 2017
@lunny

lunny commented Mar 18, 2017

Copy link
Copy Markdown
Member

maybe some screenshots?

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 18, 2017
@strk

strk commented Mar 18, 2017 via email

Copy link
Copy Markdown
Member Author

@pgaskin

pgaskin commented Mar 19, 2017

Copy link
Copy Markdown
Contributor

Maybe some screenshots so we can see the nice captchas? 😃 LGTM

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 19, 2017
@lunny

lunny commented Mar 20, 2017

Copy link
Copy Markdown
Member

LGTM except @geek1011 's comment.

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 20, 2017
@strk

strk commented Mar 20, 2017

Copy link
Copy Markdown
Member Author

@bkcsoft can you merge now ? :)

captcha

@bkcsoft
bkcsoft merged commit 97ee889 into go-gitea:master Mar 20, 2017
@strk
strk deleted the openid-captcha branch March 20, 2017 08:04
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants