mpfmorawski / git-to-know-me

3 stars 1 forks source link

PR serwisu autoryzacji #12

Closed lapankrz closed 2 years ago

lapankrz commented 2 years ago

Zgodnie z sugestią Pawła.

pktiuk commented 2 years ago

Na ten moment wszystko wydaje się ok, ale, jako że to draft to przejrzałem pobieżnie.
Porządne Review postaram się zrobić jak już będzie to w stanie nadającym się do zmergowania

mpfmorawski commented 2 years ago

@pktiuk, co rozumiesz poprzez stan nadający się do zmergowania?

pktiuk commented 2 years ago

@pktiuk, co rozumiesz poprzez stan nadający się do zmergowania?

Czyli wtedy, kiedy przestanie to być draft.
Teraz to jest draft zmian.

To, że kod jest niegotowy widać w takich miejscach jak to:

    def verify_session(self, model: SessionData) -> bool:
        """If the session exists, it is valid"""
        return True 

@lapankrz Tak BTW bardzo podoba mi się podoba to, że używasz adnotacji i dokumentujesz kod