castsbrasil / castsbrasil

Casts com conteúdo em português totalmente grátis!
http://castsbrasil.com/
Apache License 2.0
141 stars 66 forks source link

Remover autenticação via cadastro #152

Closed mfbmina closed 9 years ago

mfbmina commented 9 years ago

Pessoal, numa conversa com o @plribeiro3000 em alguma das issues ele disse que estava pensando sobre tirar autenticação via cadastro e deixar apenas via github.

Acho a ideia valiosa pois com isso iremos minimzar a quantidade de fakes e de lixo postado no sistema.

O que acham?

OBS: Esqueci dos labels :( favor adicionar

lucasldemello commented 9 years ago

Acho interessante a idéia de remover o cadastro afinal o publico do projeto é pessoal que vai ter ou já tem uma conta no github.

Em 22 de outubro de 2014 14:16, Matheus Mina notifications@github.com escreveu:

Pessoal, numa conversa com o @plribeiro3000 https://github.com/plribeiro3000 em alguma das issues ele disse que estava pensando sobre tirar autenticação via cadastro e deixar apenas via github.

Acho a ideia valiosa pois com isso iremos minimzar a quantidade de fakes e de lixo postado no sistema.

O que acham?

— Reply to this email directly or view it on GitHub https://github.com/RubyCastsBrasil/RubyCastsBrasil/issues/152.

brunoocasali commented 9 years ago

Sobre as labels eu acho que só quem faz parte do team consegue adicionar! UHEUAHEUAEUAHEU./ Sou favor de deixar só a auth com GitHub ;D

juhguu commented 9 years ago

Eu entendo que ia tirar um pouco de "peso" das nossas costas, mas não concordo em tirar o cadastro "nativo" da ferramenta. A idéia é tornar a ferramenta acessível a maior quantidade de pessoas, e isso inclui quem está iniciando e não sabe se quer o que é um controle de versão ou uma ferramenta como o github. Obrigar que o cara tenha um cadastro numa ferramenta apenas para poder acessar outra ferramenta é um tiro no pé, sem contar que as vezes nós simplesmente não queremos vincular contas. Eu por exemplo, acho mais organizado manter tudo separado

brunoocasali commented 9 years ago

Ótimo ponto @juhguu , mas a intenção de que quem se cadastrasse era a possibilidade de cadastrar screencasts certo?

juhguu commented 9 years ago

Até onde eu sei, não. O cadastro será pré-requisito também para assistir os vídeos. Sem contar que nas próximas versões estão cogitando gamificar a plataforma e implementar outros recursos, o que certamente abriria possibilidades até para os que não postam screencasts.

lbernardelli commented 9 years ago

Eu concordo com @juhguu. Acho importante dar acesso ao máximo de pessoas. O intuito é ter colaboração e com isso não podemos nos fechar, temos que acreditar que as pessoas venham contribuir da melhor forma.

brunoocasali commented 9 years ago

Pra mim a resposta à essa issue é: tudo depende do que o usuário poderá fazer. Se ele só poderá cadastrar screencasts logado => GitHub Se ele depender de uma conta para assistir => Email normal.

Eu não me lembro de ter lido sobre isso de assistir os vídeos precisará de login, vou verificar...

mfbmina commented 9 years ago

:+1: @brunoocasali

pragmaticivan commented 9 years ago

Precisamos do usuário logado para contabilizar e gerar estatísticas de uso. Da pra tirar agora não isso. E no futuro vamos precisar fazer gameficação, e de qualquer forma, essa funcionalidade já está pronta, estamos apenas aplicando o design em breve. Vai ser bem interessante deixar isso pra no futuro poder adicionar mais outras integrações, como por exemplo facebook e bitbucket (opcionalmente) sem precisar alterar muita coisa no código.

mfbmina commented 9 years ago

@pragmaticivan então cara.. mas essas outras integrações já foram até retiradas e acho que foram por mim até..

O foco é manter a forma de entrar no sistema e autenticar de apenas uma maneira que evitaria um número mais de fakes e de lixo postado na plataforma

pragmaticivan commented 9 years ago

Sim, como falei "opcionalmente" não quer dizer que vai ter. Mas da forma que está agora existe a fácil possibilidade de integrar com o que quiser. Principalmente quando se trata de adicionar mais dados e afins.

lcezermf commented 9 years ago

Então o que fica decidio, fica como está ? Isso se relaciona com o issue que to fazendo #144 queria saber p/ não codar coisa a mais ou a menos.

pragmaticivan commented 9 years ago

Continua fazendo.

plribeiro3000 commented 9 years ago

@lccezinha Toca o barco, ainda tem tempo pra decidir isso aqui. São 7 dias ou caso cheguemos a uma conclusão que no caso ainda não temos.

Eu acho que o que o @brunoocasali falou é interessante. Mantemos os dois logins, mas o cara só consegue postar casts se tiver o github autenticado na plataforma.

@pragmaticivan Não estamos falando de remover nada. Só de mudar o requisito para postar casts de forma que possamos controlar melhor os spams.

Como o @mfbmina comentou a gente removeu a quantidade de autenticações para simplificar. Vide aqui. Acredito que isto vai somente estar continuando o que já foi decidido e nos dar a opção de ter algo para diminuir/erradicar os spams.

pragmaticivan commented 9 years ago

:+1:

mfbmina commented 9 years ago

Antes minha ideia era realmente remover. Porem vendo os pontos de vista do @plribeiro3000 e do @brunoocasali eu sou obrigado a concordar com eles.

plribeiro3000 commented 9 years ago

@mfbmina :+1:

plribeiro3000 commented 9 years ago

@juhguu e @lbernardelli que optaram por não remover a autenticação por email, o que acham da solução dada pelo @brunoocasali ?

lcezermf commented 9 years ago

Fmz, amanhã tento entregar essa issue, valeu !

plribeiro3000 commented 9 years ago

@lccezinha :star:

plribeiro3000 commented 9 years ago

Ae @pragmaticivan @juhguu @lbernardelli vocês estão de acordo com a nova solução?

pragmaticivan commented 9 years ago

concordo :dart:

ribeiroevandro commented 9 years ago

Certo, entendo o que o público seja desenvolvedor ou quem quiser começar e que ter uma conta no Github é o básico, mas eu por exemplo uso uma conta de email apenas para esse tipo de cadastro, centralizar as notificações. Acho válido a permanencia do cadastro por email, poderiamos ter alguma rotina que checasse isso, caso determinada conta fique inativa por muito tempo, seja removida. Evitaria os fakes na base.

mfbmina commented 9 years ago

@ribeiroevandro pelo que entendi você possui uma conta de email para cadastros certo? a sua conta do github não está nesse email para cadastros? qual seria a diferenca? Apenas queremos que um cara que vá postar um conteudo, tenha uma conta do github tambem, pra tentar evitar que ele seja um spammer ou um cara que vai postar lixo apenas por postar.

pragmaticivan commented 9 years ago

@mfbmina isso já foi concordado, pra postar algo precisa ter conta no github, porém pra acessar conteúdo, apenas a de email resolve.

pragmaticivan commented 9 years ago

Acho que teve outra thread falando sobre a necessidade de estar logado para assistir screencast. Não lembro bem qual foi.

mfbmina commented 9 years ago

@pragmaticivan saquei.. achei que ele tava falando da thread sobre postar e não sobre assistir! Falha minha :disappointed:

ribeiroevandro commented 9 years ago

@mfbmina No meu caso a conta do github é a mesma que uso para cadastrar notificações, mas com esse pensamento, de ter uma conta para isso, fiz a sugestão, outros podem ter a mesma forma de organizar, porém ter criado a conta antes usando outro email.

plribeiro3000 commented 9 years ago

Galera. Vamos tentar manter o foco pra não dispersar muito. Temos uma nova solução que pretende atender aos requisitos das duas opiniões. Se tiver algo a favor/contra comentem, se não for relacionado, abram outra discussão. Obrigado!

Vou deixar isso aberto até amanhã. Se os que descordaram inicialmente não se pronunciarem, vou considerar isso como decidido.

OBS: Essa discussão foi umas das primeiras com iteração dessa forma e achei muito fera por que no final conseguimos chegar a uma solução que não atingisse nenhum dos lados. Muito obrigado a quem participou!

plribeiro3000 commented 9 years ago

@ribeiroevandro Agora que entendi o que quis dizer. ehhehe

Não acho que uma task no background seja suficiente para barrarmos os spammers. Ainda mais considerando que estamos usando contas de hosting free e esse tipo de coisa dispensa mais recursos do servidor. Se pudermos confirmar que o cara é um usuário real e que o usuário não tem permissão para zuar o sistema (o esquema de pontos é pra isso) já estaremos tendo um controle muito bom sobre isso. =)

Só resta saber a melhor forma de fazermos isso, mas isso ja é pra outra discussão.

plribeiro3000 commented 9 years ago

Fechando essa discussão. Obrigado a todos que participaram!