FriendsOfSymfony / friendsofsymfony.github.io

Description of what FriendsOfSymfony (aka FoS) is about.
http://friendsofsymfony.github.io
32 stars 4 forks source link

New maintainer for FOSFacebookBundle #34

Closed diegoholiveira closed 11 years ago

diegoholiveira commented 12 years ago

There is a lot of PR waiting for revision on the bundle repository, and since I'm using this bundle a lot in a project, I can be the maintainer for this bundle (it's just a suggestion, can be someone else).

For me it's important keep this bundle updated.

How can we proceed to choose someone to be in charge of this bundle?

lsmith77 commented 12 years ago

Right now we do not have any member of FOS looking after the facebook and twitter bundles. especially the facebook one has PRs piling up and @diegoholiveira seems to be one of the most active people. so i am +1 to give him access to at least the facebook bundle.

arghav commented 12 years ago

:+1:

asm89 commented 12 years ago

Since the bundle is brought up here, maybe we should consider deprecating/removing the login functionality of the bundle? I see a lot of questions coming by on for example IRC of people using the bundle solely to login on facebook. Those people are probably better of using HWIOAuthBundle.

merk commented 12 years ago

I am +1 for anything that will end up increasing activity of the FOS bundles and reduce the support questions occuring in #symfony.

Are you willing to increase the documentation quality as well as maintain the code, @diegoholiveira?

lsmith77 commented 12 years ago

@asm89 yeah .. makes sense to me as well.

diegoholiveira commented 12 years ago

@merk: about improve the docs, sure that I'll work on it.

@asm89, @lsmith77: FOSFacebookBundle are just about Login. If we deprecate the login functionality, it's better deprecate all bundle, no? I take a look on HWIOAuthBundle and it seems to be an excellent choice to provide login functionality on Facebook and another services.

Diego Oliveira Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Friday, September 21, 2012 at 4:08 AM, Lukas Kahwe Smith wrote:

@asm89 (https://github.com/asm89) yeah .. makes sense to me as well.

— Reply to this email directly or view it on GitHub (https://github.com/FriendsOfSymfony/friendsofsymfony.github.com/issues/34#issuecomment-8756780).

arghav commented 11 years ago

This case applies to TwitterBundle too. Since the main use of both these bundles is to integrate with Symfony security it might not make sense to maintain these two anymore. For using the specific API it's just a matter of creating a service.

schmittjoh commented 11 years ago

The last time I checked HwiOauthBundle was not so extensible because it provides integration for a lot of different providers.

Personally, I'm still using the TwitterBundle.

On Fri, Sep 21, 2012 at 2:58 PM, Amal Raghav notifications@github.comwrote:

This case applies to TwitterBundle too. Since the main use of both these bundles is to integrate with Symfony security it might not make sense to maintain these two anymore. For using the specific API it's just a matter of creating a service.

— Reply to this email directly or view it on GitHubhttps://github.com/FriendsOfSymfony/friendsofsymfony.github.com/issues/34#issuecomment-8763666.

asm89 commented 11 years ago

The bundle indeed focusses on providing oauth1/2 login integration and provides a lot of different providers that are pre configured.

diegoholiveira commented 11 years ago

Like @schmittjoh, I'll keep using FOSFacebookBundle. But I guess we need to decide if we'll keep it or not. And if we decide to keep, I can be in charge of it.

On Friday, September 21, 2012 at 11:21 AM, Alexander wrote:

The bundle indeed focusses on providing oauth1/2 login integration and provides a lot of different providers that are pre configured.

— Reply to this email directly or view it on GitHub (https://github.com/FriendsOfSymfony/friendsofsymfony.github.com/issues/34#issuecomment-8766091).

stof commented 11 years ago

@diegoholiveira FOSTwitterBundle is providing more stuff than just the OAuth login.

lsmith77 commented 11 years ago

i think its clear that there is interest in the Bundle .. even just "legacy" interest. same applies to FOSTwitterBundle. so imho we should just add @diegoholiveira as he is willing to step up.

lsmith77 commented 11 years ago

so atm we have a +1 from @merk, @kertz ... anyone else?

jmikola commented 11 years ago

:+1:

khepin commented 11 years ago

:+1:

asm89 commented 11 years ago

:+1: I still hope we can figure something out on the authentication part though. Quite some users seem to want login with social providers, not only facebook. I still see a lot of questions coming by about this on IRC. HWIOAuthBundle is not perfect yet, but I've been working hard with @stloyd to make using the bundle for authentication as effortless as possible, as extra it already provides FOSUserBundle integration out of the box.

@diegoholiveira Let me know what you think. :)

diegoholiveira commented 11 years ago

@asm89 I do agree in deprecate this bundle in favor of HWIOAuthBundle.

I'll keep to using FOSFacebookBundle because it's already working on my project and I can't migrate to HWIOAuthBundle now.

If we decide to deprecate it, we still need to choose a new maintainer of the bundle?

Notice: I'm not saying that we need to deprecate FOSFacebookBundle, but if we do it, I'll not be against it.

schmittjoh commented 11 years ago

+1

lsmith77 commented 11 years ago

ok .. we are still not at half of the current members .. but i decided to just do ahead an add @diegoholiveira into the FOS Owner group in this repo as well as the FOSFacebookBundle team in the respective repo.

diegoholiveira commented 11 years ago

Thanks @lsmith77, I'll do my best on FOSFacebookBundle :)

stof commented 11 years ago

@lsmith77 the repo teams are useless since the refactoring of the github notification system. Previously, orga owners were not notified without being in a team with write access, so we created these teams. Now, you choose on a per repo basis (and it is activated by default when you have write access, which includes owners)