Fearless refactoring 2.0

lizardsystem / lizard-auth-server

Django backend for the old SSO server

http://lizard-auth-server.readthedocs.io/

MIT License

2 stars 1 forks source link

Fearless refactoring 2.0 #52

Closed reinout closed 8 years ago

reinout commented 8 years ago

Botte bijl editie! Alle nieuwe modellen eruit. Oude modellen weer gaan gebruiken, maar zonder alle connecties ertussen.

Nieuw /api/v2/ beginpunt waarin de URLs staan zodat ik die kan veranderen. Dus.... gelijk de naamgeving maar opgeschoond.

De docstrings zijn van adembenemende kwaliteit. Zelfs de sphinx documentatie is bijgewerkt (en vooral uitgedund, zucht).

Belangrijk: lizard-auth-client moet nog verbouwd worden!

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling ef7f6ce28dde81f31a12fe9f03cd22975c1b4764 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling 9983500c5d7420606551f757204ee223e6195750 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

reinout commented 8 years ago

@byrman, kun je qua code alvast kijken?

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling c8c3c36a457b279adeb87ebbdc43c1e3ec08ce03 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

byrman commented 8 years ago

Een flinke verbouwing. Weet je al hoe het endpoint voor het aanmaken van een nieuwe gebruiker eruit gaat zien? Of gaat dat via lizard-auth-client lopen?

reinout commented 8 years ago

Aanmaken nieuwe gebruiker: dat zal wel /api/v2/new_user/ worden. Dat maakt dan een nieuwe gebruiker aan (of geeft een bestaande gebruiker terug op basis van emailadres).

Maar.... Eerst wat er nu qua jwt/v2 api was weer aan de praat krijgen in de nieuwe situatie. Ik ben met lizard-auth-client bezig. Als dat weer werkt en op staging staat, dan kan ik nieuw spul toevoegen.

Tenminste, dat lijkt me de beste weg?

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling dce69deeec3e1b40f52b762367f41212ce7ba121 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

reinout commented 8 years ago

Ik kan trouwens ook wel even langslopen om alles uit te leggen. Aan de andere kant is het zelf-doorlezen een goede test of de docstrings en de variabelenamen duidelijk zijn.

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling ac9f3ce8e56906fee81b8f6ab953d5228bd5a3a9 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling ac9f3ce8e56906fee81b8f6ab953d5228bd5a3a9 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling e1e52751bb464bbc242c40ec9701d3030b592332 on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.

coveralls commented 8 years ago

Coverage decreased (-2.6%) to 77.38% when pulling 8dad941080d9f6ab4787e1e4f2a006d2a0712acb on reinout-v2-2.0 into 9711e8ce74289c4a13150264552b8a0e6677a3f5 on master.