search

AlessandroStaffolani / shamir-fido

Project for a simple electron application that uses shamir's secret sharing and FIDO 2-factor-authentication
MIT License
1 stars 0 forks source link

Did mount #4

Closed sabbio93 closed 5 years ago

sabbio93 commented 5 years ago

https://github.com/ale8193/shamir-fido/blob/22aee30a22a273df8cc363018a195392259ab7cd/src/App.js#L22 Come mai hai messo l'inizializzazione in didmount invece che nel constructor

AlessandroStaffolani commented 5 years ago

Inizialmente avevo intenzione di fare anche altre operazioni nel didmount ma poi ho notato che era meglio metterle nel didUpdate e quella l'ho lasciata li, alla fine non cambia nulla, però forse è più elegante toglierlo e metterlo nel costruttore

AlessandroStaffolani commented 5 years ago

In ogni caso per ho rimosso il set state nel didmount e l'ho messo nel costruttore

sabbio93 commented 5 years ago

@ale8193 Anche il didupdate l hai tolto?

AlessandroStaffolani commented 5 years ago

No no il didupdate l'ho lasciato perchè serve