Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

inokufu/ralph

thoumasse pushed 3 commits to feature/cozy_stack_backend inokufu/ralph
  • ✨(backends) add CozyStack data and lrs backends 4b23d52
  • 🔨(test) set up debugpy in conftest c9f5322
  • 🚧(tests) add a python script to test cozy backend fdc68f5

View on GitHub

thoumasse pushed 3 commits to feature/cozy_stack_backend inokufu/ralph
  • ✨(backends) add CozyStack data and lrs backends 33dbb9e
  • 🔨(test) set up debugpy in conftest d35ac18
  • 🚧(tests) add a python script to test cozy backend 2db03f8

View on GitHub

thoumasse pushed 4 commits to feature/cozy_stack_backend inokufu/ralph
  • 🛂(auth) add Cozy authentication methods 5a504f9
  • ✨(backends) add CozyStack data and lrs backends 76fe98c
  • 🔨(test) set up debugpy in conftest cda7f74
  • 🚧(tests) add a python script to test cozy backend b362122

View on GitHub

thoumasse pushed 4 commits to feature/cozy_stack_backend inokufu/ralph
  • 🛂(auth) add Cozy authentication methods 358e942
  • ✨(backends) add CozyStack data and lrs backends 3f9cd24
  • 🔨(test) set up debugpy in conftest e7272ec
  • 🚧(tests) add a python script to test cozy backend a2e0bcb

View on GitHub

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

View on GitHub

pierrocknroll created a review on a pull request on inokufu/ralph

View on GitHub

thoumasse created a review comment on a pull request on inokufu/ralph
Bien vu

View on GitHub

thoumasse created a review on a pull request on inokufu/ralph

View on GitHub

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

View on GitHub

thoumasse created a review on a pull request on inokufu/ralph

View on GitHub

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...

View on GitHub

thoumasse created a review on a pull request on inokufu/ralph

View on GitHub

pierrocknroll created a review comment on a pull request on inokufu/ralph
Peut-être faudrait-il mettre cozysessid dans une constante quelque part

View on GitHub

pierrocknroll created a review comment on a pull request on inokufu/ralph
Peut-être extraire cela dans une méthode séparée.

View on GitHub

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...

View on GitHub

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.

View on GitHub

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

View on GitHub

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

View on GitHub

thoumasse pushed 3 commits to feature/cozy_stack_backend inokufu/ralph
  • ✨(backends) add CozyStack data and lrs backends adbb6c8
  • 🔨(test) set up debugpy in conftest 859a99e
  • 🚧(tests) add a python script to test cozy backend 0320f5f

View on GitHub

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 pushed 4 commits to feature/cozy_stack_backend inokufu/ralph
  • 🛂(auth) add Cozy authentication methods a1b1f94
  • ✨(backends) add CozyStack data and lrs backends acf22e0
  • 🔨(test) set up debugpy in conftest e38d4b7
  • 🚧(tests) add a python script to test cozy backend e73b7e3

View on GitHub

thoumasse created a branch on inokufu/ralph

feature/cozy_stack_backend - gear: Ralph, the ultimate Learning Record Store (and more!) for your learning analytics