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

Already on GitHub? Sign in to your account

Change layout & Meetup auth #1

Closed
wants to merge 25 commits into
from

Conversation

Projects
None yet
5 participants

Nouveau layout avec proposition de talks via son compte meetup.

Contributor

pi3r commented Mar 19, 2012

Je vais y jeter un coup d'oeil dès que j'ai le temps.

Owner

tibastral commented Mar 20, 2012

ok great, je t'avoue que j'ai pas vraiment le temps en ce moment !
En tout cas, Remink, génial !! Merci pour ces commits !!

@tibastral tibastral and 1 other commented on an outdated diff Mar 20, 2012

app/views/presentations/index.html.haml
@@ -0,0 +1,22 @@
+%h2 Appel à présentations : proposez un talk !
+%p
+ Faites vos propositions de talks pour le prochain
+ %strong
+ Paris.rb
+ , qu'on sélectionne et qu'on organise tout ça.
+
+#{render '/presentations/new'}
@tibastral

tibastral Mar 20, 2012

Owner

= render '/presentations/new'
Non ?

@remivigan

remivigan Mar 20, 2012

Effectivement, pas très familier avec HAML ;-)

@tibastral

tibastral Mar 23, 2012

Owner

Super, bah dès que tout ça est ok, on merge et on déploie !

Thibaut Assus
Founder & consultant @ http://milesrock.com
Organizer @ http://meetup.rubyparis.org
06 24 57 14 88
Specialized in ROR / Node.js
“Programming at its best is an act of empathy.” - Kent Beck

On Tuesday, March 20, 2012 at 6:24 PM, remink wrote:

@@ -0,0 +1,22 @@
+%h2 Appel à présentations : proposez un talk !
+%p

  • Faites vos propositions de talks pour le prochain
  • %strong
  • Paris.rb
  • , qu'on sélectionne et qu'on organise tout ça.
    +
    +#{render '/presentations/new'}

Effectivement, pas très familier avec HAML ;-)


Reply to this email directly or view it on GitHub:
https://github.com/tibastral/presentations/pull/1/files#r581146

@tibastral tibastral and 1 other commented on an outdated diff Mar 20, 2012

test/fixtures/events.yml
@@ -0,0 +1,13 @@
@tibastral

tibastral Mar 20, 2012

Owner

faudrait virer ces fichiers inutiles

@remivigan

remivigan Mar 20, 2012

Agree.
Il faut les virer ou faire des tests, ce qui serait bien mieux ! ;-)

@tibastral tibastral and 1 other commented on an outdated diff Mar 20, 2012

public/assets_old/application.css
@@ -0,0 +1 @@
+.wrapper{font-family:Helvetica neue,sans-serif;margin:0 auto;overflow:hidden;*zoom:1}.wrapper .presentation-el{font-size:20px}input[type=text],input[type=email]{font-size:30px}input[type=submit]{font-size:40px;color:green;float:right}
@tibastral

tibastral Mar 20, 2012

Owner

Ce serait cool de supprimer ces fichiers plutôt que de les mettre ici !

@remivigan

remivigan Mar 20, 2012

Agree.
C'était mis de coté au cas où, mais visiblement cela n'a aucun intérêt.

@simonc simonc and 1 other commented on an outdated diff Mar 20, 2012

app/controllers/application_controller.rb
@@ -1,3 +1,24 @@
class ApplicationController < ActionController::Base
protect_from_forgery
+
+ before_filter :last_tweets, :check_new_event
+ helper_method :current_user
+
+
+ def last_tweets
+ @tweets = Tweet.order("created_at DESC").limit(5).all
+ end
+
+ def check_new_event
+ if Event.last.date > Time.now.to_date
@simonc

simonc Mar 20, 2012

Il faudrait vérifier l'existance d'events avant de faire un appel à date.

Après un déploiement, l'application ne démarre pas faute d'event présent.
Une bonne pratique consisterait d'ailleur à avoir un Event.last_date ou similaire :)

@remivigan

remivigan Mar 23, 2012

Si seulement j'avais accompagné tout ça de tests :)
Merci Simon!

Des commit ont été ajoutés afin de corriger les erreurs remontées en commentaires.

Owner

tibastral commented Mar 26, 2012

Hello Remink !

Ton travail est très bon, j'ai encore quelques petits problèmes à l'intégrer (et n'aurai pas le temps de faire les modifs d'ici le prochain event, je pars en vacances entre temps).

  • est-il possible d'aller taper dans meetup directement pour les nouveaux events (import ?)
  • le twitter n'est pas utile pour le moment (j'ai toujours pas récup @parisrb)
  • la page de contact, tu peux mettre mon mail milesrock et mon twitter en attendant mieux !

En tous cas, super boulot !

Thibaut Assus
Founder & consultant @ http://milesrock.com
Organizer @ http://meetup.rubyparis.org
06 24 57 14 88
Specialized in ROR / Node.js
“Programming at its best is an act of empathy.” - Kent Beck

On Saturday, March 24, 2012 at 1:50 PM, remink wrote:

Des commit ont été ajoutés afin de corriger les erreurs remontés en commentaires.


Reply to this email directly or view it on GitHub:
#1 (comment)

Contributor

jarthod commented Nov 9, 2012

Du nouveau sur ce merge ? je voudrait participer pour améliorer le design, en partant d'une version propre et à jour ;)

Contributor

pi3r commented Nov 13, 2012

Je pense que cette pull request ne sera jamais mergée, étant donné qu'elle ne merge pas automatiquement.

Contributor

jarthod commented Nov 13, 2012

C'est dommage de laisser tomber tout ce travail de remink non ? Si c'est juste la flemme de merge je veux bien le faire ^^

Contributor

pi3r commented Nov 13, 2012

A voir avec Thibaut, j'ai pas du tout suivi les modifications apportées par cette PR

Contributor

jarthod commented Nov 14, 2012

ping @tibastral ?

Owner

tibastral commented Nov 14, 2012

pong

On Wednesday, November 14, 2012 at 5:26 PM, Adrien Jarthon wrote:

ping @tibastral (https://github.com/tibastral) ?


Reply to this email directly or view it on GitHub (#1 (comment)).

Contributor

pi3r commented Dec 11, 2012

Can we close this one since it is not mergeable and too big to read ?

@tibastral tibastral closed this Dec 11, 2012

@tibastral tibastral reopened this Feb 6, 2013

Owner

tibastral commented Feb 6, 2013

Adrien, tu peux merger dans ta branche, et je te propose qu'on s'appelle pour discuter de ce qu'on peut garder / enlever.

Contributor

jarthod commented Feb 8, 2013

J'ai pas encore merge mais j'ai fais la listes des modifications apportés:

  • Un nouveau modèle User et controlleur Sessions avec OAuth sur meetup.com
  • Un nouveau layout avec une page d'accueil, une page events, des pages présentations (CRUD), une page de contact
  • Un nouveau CSS pour tout le site
  • Ajout des traduction standard rails fr
  • Un modèle Tweet avec un tache pour les récupérer

Que penses-tu qu'on devrait garder ? je m'occuperai de l'intégrer dans la version actuelle.

Owner

tibastral commented Feb 8, 2013

On Friday, February 8, 2013 at 1:02 PM, Adrien Jarthon wrote:

J'ai pas encore merge mais j'ai fais la listes des modifications apportés:
Un nouveau modèle User et controlleur Sessions avec OAuth sur meetup.com (http://meetup.com)

Je pense que ça peut être intéressant, mais à voir si ça complexifie pas trop le workflow. Au niveau du workflow :
je veux proposer un talk, pour l'instant, je laisse mon adresse email, mon nom de talk et mon nom, ma durée et mon niveau
On peut mettre un petit login with meetup qui fetche mon id utilisateur, et qui me permet de proposer un talk avec mes infos préfetchées

Un nouveau layout avec une page d'accueil, une page events, des pages présentations (CRUD), une page de contact

Le nouveau layout, je suis pas trop fan. et la page events, ça fait doublon avec le meetup donc clairement non. La page de contact ne sert pas à grand chose non plus. Il faudrait en revanche une page pour sponsoriser.

Un nouveau CSS pour tout le site

Why not, mais il faut pas que ce soit trop lourd comme design, le style graphique a déjà pas mal évolué, et on va avoir un nouveau logo bientôt, qui va remettre tout à plat :)

Ajout des traduction standard rails fr

Ça peut être intéressant, d'autant que le site est en Français (peut-être faire une version anglaise pourrait avoir du sens)

Un modèle Tweet avec un tache pour les récupérer

Je vois pas trop l'intérêt perso,

Que penses-tu qu'on devrait garder ? je m'occuperai de l'intégrer dans la version actuelle.

Dans l'absolu, si c'était des feature branch avec des trucs super précis, on pourrait discuter les features dan les différentes PR. Là c'est clairement trop monolithique pour le moment.
Bisous


Reply to this email directly or view it on GitHub (#1 (comment)).

Contributor

jarthod commented Feb 8, 2013

Ok,

Dans ce cas je vais essayer de faire une pull request avec juste le login meetup (sans empêcher le workflow normal) et les traductions.
Pour le CSS de toute façon si on prend pas le nouveau layout il fonctionnera pas.
Je suis d'accord pour le twitter, on peut juste mettre un widget twitter avec le hashtag de toute façon.

@tibastral tibastral closed this Jun 24, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment