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

Generic with authors #121

Merged
merged 3 commits into from
Apr 15, 2016
Merged

Generic with authors #121

merged 3 commits into from
Apr 15, 2016

Conversation

cabelotaina
Copy link
Member

Nesta proposta o generic-theme recebe a contribuição de páginas para os usuários que já esta no creta, além disso fiz algumas melhorias na url criando um hash, já que o vota cultura possui usuários que o nome de login é o email do usuário.

Algumas pendencias:

  • As urls que resultantes de usuários mencionados nos comentários não estão cobertas, e atualmente nem estão funcionando.
  • O hash usado nas urls precisa de duas chaves que estão publicas no github, o que permite que qualquer usuário conheça como decriptar os id dos usuários do site, para evitar incomodos essas chaves poderiam ser definidas na pagina de configurações pelo admin do delibera. Acredito que essa é uma boa saída, o que acha @jacsonp?

@lpirola
Copy link

lpirola commented Apr 14, 2016

Alto nível esse PR!!!!! 🌟🌟🌟🌟

@jacsonp
Copy link
Member

jacsonp commented Apr 15, 2016

Não rola usar umas das chaves da configuração para criptografar ou para gerar essas chaves?
São defines, de ser fácil de acessar como constantes:
AUTH_KEY: Added to ensure better encryption of information stored in the user's cookies.
SECURE_AUTH_KEY: Added to ensure better encryption of information stored in the user's cookies.
LOGGED_IN_KEY: Added to ensure better encryption of information stored in the user's cookies.
NONCE_KEY: Added to ensure better encryption of information stored in the user's cookies.
AUTH_SALT: Used to make the AUTH_KEY more secure.
SECURE_AUTH_SALT: Used to make the SECURE_AUTH_KEY more secure.
LOGGED_IN_SALT: Used to make the LOGGED_IN_KEY more secure.
NONCE_SALT: Used to make the NONCE_KEY more secure.

Ou se não gerar random e armazenar como configuração do blog/projeto

@@ -32,7 +32,7 @@
</select>
</form>
<a href="<?php echo get_site_url(); ?>/delibera/membros" >Ver todos os Membros</a>
Copy link
Member

Choose a reason for hiding this comment

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

Não tem a ver com o seu PR em si, apenas ao ler lembrei que tínhamos que arrumar esse /delibera, fazer como os outros plugin, ou usar uma função que retorna a url e outra que retorna o dir, ou definir contantes para evitar problemas com o nome da pasta.

Copy link
Member Author

Choose a reason for hiding this comment

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

Opa que massa seu comentário acabou me mostrando que não coloquei a função de tradução ali no "Ver todos os membros" rs.

@jacsonp
Copy link
Member

jacsonp commented Apr 15, 2016

Ta muito bom mesmo aqui, não aceitei o merge só por causa da chave, vamos ajeitar isso e daí ta ótimo

@jacsonp jacsonp merged commit 91395b7 into generic-theme Apr 15, 2016
@jacsonp jacsonp deleted the generic_with_authors branch April 15, 2016 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants