Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
thoumasse pushed 3 commits to feature/cozy_stack_backend inokufu/ralph
thoumasse pushed 3 commits to feature/cozy_stack_backend inokufu/ralph
pierrocknroll created a review comment on a pull request on inokufu/ralph
J'ai tendance à faire du Defense Programming, je crois fort en la loi de Murphy :D
thoumasse created a review comment on a pull request on inokufu/ralph
Il me semble que toutes les réponses doivent contenir ces champs mais ça ne coûte rien de les vérifier
thoumasse created a review comment on a pull request on inokufu/ralph
Yes je pensais faire une classe pour encapsuler toutes ces opérations dans des méthodes avec un plus haut niveau d'abstraction type `get_all_doctypes` etc. J'en profiterai pour en faire une variabl...
pierrocknroll created a review comment on a pull request on inokufu/ralph
Peut-être faudrait-il mettre cozysessid dans une constante quelque part
pierrocknroll created a review comment on a pull request on inokufu/ralph
Peut-être extraire cela dans une méthode séparée.
pierrocknroll created a review comment on a pull request on inokufu/ralph
C'est un choix d'architecture mais ta classe CozyStackDataBackend est très couplée à CozyStackClient, peut-être faudrait-il abstraire un peu (pour les tests notamment), que CozyStackClient soit une...
pierrocknroll created a review comment on a pull request on inokufu/ralph
Il faudra gérer les cas d'erreurs où ces valeurs n'existent pas dans json_response.
pierrocknroll created a review on a pull request on inokufu/ralph
Bien joué, c'est bien organisé et tu as su réutiliser ce qui est prévu dans Ralph sans trop le modifier, c'est cool. Je t'ai mis quelques retours de pinaillages
pierrocknroll created a review on a pull request on inokufu/ralph
Bien joué, c'est bien organisé et tu as su réutiliser ce qui est prévu dans Ralph sans trop le modifier, c'est cool. Je t'ai mis quelques retours de pinaillages
thoumasse pushed 3 commits to feature/cozy_stack_backend inokufu/ralph
thoumasse opened a draft pull request on inokufu/ralph
Feature/cozy stack backend
## Purpose Implement authentication against CozyStack and backends to read from and write to CozyStack database.thoumasse created a branch on inokufu/ralph
feature/cozy_stack_backend - gear: Ralph, the ultimate Learning Record Store (and more!) for your learning analytics