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

watch preselection #91

Merged
merged 21 commits into from
Jan 7, 2016
Merged

watch preselection #91

merged 21 commits into from
Jan 7, 2016

Conversation

MathieuNls
Copy link
Member

#36

@MathieuNls MathieuNls added this to the Tw 1.3 milestone Jan 4, 2016
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 4, 2016 21:19 Active
@MathieuNls MathieuNls changed the title 36 preselect watch preselection Jan 4, 2016
@AlphonseJr
Copy link
Collaborator

Alors, j'ai crée un nouveau compte (d'ailleurs, y'a pas un trick pour éviter d'avoir à créer à chaque PR un nouveau compte ?), j'ai ajouté une nouvelle montre (/new-watch/) et là écran blanc (add_watch), je suis bloquaide :(

Sinon, autre chose, y'a pas moyen d'avoir l'url dédiée à cette PR présente sur le ticket automatiquement ?

@MathieuNls
Copy link
Member Author

Sur le ticket; y'a un MathieuNls referenced this issue an hour ago Open watch preselection #91

Pour le compte, c'est pck la db a été wipe récemment, normalement t'as pas à le refaire.

@MathieuNls
Copy link
Member Author

Moi ca fonctionne, mis à part le print de debug que j'ai oublié d'enlever

screenshot from 2016-01-04 17 10 16

@AlphonseJr
Copy link
Collaborator

Je me suis mal exprimé, je voulais dire le https://tw-prepod-pr-xx.herokuapp.com/ pour ne pas avoir à rentrer l'url à la main à chaque fois !

@MathieuNls
Copy link
Member Author

Il y est ici en fait, dans la PR, puisque le déploiement est lié à la PR, pas à l'issue. Check y'a un squirrel avec deployed qui t'y envoi direct.

@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 4, 2016 22:14 Active
@AlphonseJr
Copy link
Collaborator

Ah trop bien ! Merci le squirel !

Par contre, j'ai toujours l'écran blanc :

capture d ecran 2016-01-04 a 23 12 47

Si ça peut aider, je suis sur Chrome Version 47.0.2526.106 (64-bit) depuis mon MacBookPro

@MathieuNls
Copy link
Member Author

Fais click droit, inspect puis va dans l'onglet console pour voir si y'a des soucis chez toi. Aussi essai de delog / relog vu que j'ai joué avec la DB

@AlphonseJr
Copy link
Collaborator

Je viens de me delog / relog et là, au lieu de faire page blanche, ca me renvoie sur la home en me délogant !

@MathieuNls
Copy link
Member Author

Je viens d essayer sur Chrome 47.0.2526.106 MacOSX 10.11.2 (El Capitan) et ca fonctionne aussi pour l'ajout.

@MathieuNls
Copy link
Member Author

Par contre; y'a un soucis dans les redirections.

Quand je click sur measures alors que je suis loggé, ca me redirige vers la home.

@MathieuNls MathieuNls changed the title watch preselection [WIP] watch preselection Jan 4, 2016
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 00:43 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 00:49 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 01:25 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 01:31 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 01:38 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 01:53 Active
@MathieuNls
Copy link
Member Author

ca a l'air plus stable là.

@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 5, 2016 16:53 Active
@AlphonseJr
Copy link
Collaborator

Toujours ce même fichu écran blanc :(

J'ai aussi du code qui s'affiche sur la home et sur le dashboard :

capture d ecran 2016-01-05 a 19 44 55

@MathieuNls
Copy link
Member Author

Tu peux essayer en clearant tes caches ou navigation privée ?

Le code c'est normal, c'est un check pour les sessions (qui causes tes soucis).

@AlphonseJr
Copy link
Collaborator

Testé à l'instant en navigation privée sur mon MBP et mon PC windows sur Chrome tous les deux et toujours cet écran blanc :(

@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 6, 2016 17:01 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 6, 2016 18:08 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 6, 2016 18:10 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 6, 2016 18:28 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 6, 2016 22:44 Active
@MathieuNls MathieuNls deployed to tw-prepod-pr-91 January 6, 2016 22:47 Active
@MathieuNls MathieuNls changed the title [WIP] watch preselection watch preselection Jan 6, 2016
AlphonseJr added a commit that referenced this pull request Jan 7, 2016
@AlphonseJr AlphonseJr merged commit b110b43 into dev Jan 7, 2016
@AlphonseJr
Copy link
Collaborator

Ca marche !! Bien joué :)

@MathieuNls MathieuNls deleted the 36_preselect branch February 15, 2016 15:44
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.

2 participants