Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

增加腾讯QQ/微信/微博客户端 #102

Closed
wants to merge 4 commits into from
Closed

增加腾讯QQ/微信/微博客户端 #102

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 16, 2016

Q A
New feature? yes
Tests pass? yes

@samdark
Copy link
Member

samdark commented Apr 16, 2016

Thanks for the effort but we aren't going to include more providers into the extension. I suggest releasing it as a separate package. After it's done I can add a link to that package to docs. Thanks!

@samdark samdark closed this Apr 16, 2016
@ghost
Copy link
Author

ghost commented Apr 16, 2016

这几个接口,中国用户使用率100%,由于GFW的问题,多个包安装是非常慢的。

@samdark samdark reopened this Apr 16, 2016
@samdark
Copy link
Member

samdark commented Apr 16, 2016

OK. We can make an exception then...

@samdark samdark added the type:enhancement Enhancement label Apr 16, 2016
@samdark samdark added this to the 2.0.6 milestone Apr 16, 2016
@samdark samdark self-assigned this Apr 16, 2016
@ghost
Copy link
Author

ghost commented Apr 16, 2016

thx

@samdark
Copy link
Member

samdark commented Apr 16, 2016

Would be great if you'll add changelog. I'll merge right after.

@ghost
Copy link
Author

ghost commented Apr 16, 2016

ok

@ghost
Copy link
Author

ghost commented Apr 16, 2016

Add change log to complete.

@samdark samdark closed this in 87a0648 Apr 22, 2016
@samdark
Copy link
Member

samdark commented Apr 22, 2016

Merged with some adjustments. Thanks!

@yujiandong
Copy link

Hello, I am the author of qq/weibo/wechat auth-client.

https://github.com/yujiandong/yii2-authclient

@ghost
Copy link

ghost commented Apr 26, 2016

@yujiandong
Do you mean you are the author of these codes instead of OP?

If it's true, maybe some further attention is needed due to copyright violation.

@samdark

@yujiandong
Copy link

@raincious @samdark
Yes, you can compare the codes https://github.com/yujiandong/yii2-authclient
I wrote it for my simpleforum 7 months ago.

But @xutongle change the author name in the file.

* @see http://open.weibo.com/
* @see http://open.weibo.com/wiki/
*
* @author Xu Tongle <xutongle@gmail.com>
Copy link

Choose a reason for hiding this comment

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

You are not author 😒

Copy link

Choose a reason for hiding this comment

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

@ghost
Copy link

ghost commented Apr 26, 2016

@yujiandong

LOL. You are not the only victim, see the 'Leaps Framework' this guy contributed to:
https://github.com/leaps/leaps/blob/3be68042b7060d2a6e4aef054efae812e57fda95/framework/Kernel.php

Looks just like:
https://github.com/yiisoft/yii2-framework/blob/master/BaseYii.php

Isn't?

Relax, there are a lot of people who take a ~$400/m full time job writing PHP code. Change the author declaration from other people's code while keep its RUNability is hard enough for them to believe they can "own" the code.

That's why they by the way changed the copyright declaration and remove you from your code as well.

Just want for the assignee make things right.

@lambdaq
Copy link

lambdaq commented Apr 26, 2016

This @xutongle guy has this badass commit

https://github.com/xutongle/yii2-authclient/commit/61528adec3f0d9ecc8ff1fa0215e75ec79869649

The commit message in Chinese reads "solving licensing problems".

I mean srsly? WTF?

@cebe
Copy link
Member

cebe commented Apr 26, 2016

Thanks for bringing this to our attention, I have removed the commit from master for now. Need some time to keep up with the comments.

@tooooolong
Copy link

@cebe 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants