refactor: session logic

[limwa]

[ATTENTION. This is the Pull Request Template. Replace the two following lines with the indicated information and delete this one. Do not delete anything else]

Closes #[issue number]
[Description of the changes proposed in the pull request. Include steps to replicate the behavior and screenshots if UI is updated]

Review checklist

• Terms and conditions reflect the current change
• Contains enough appropriate tests
• If aimed at production, writes a new summary in whatsnew/whatsnew-pt-PT
• Properly adds an entry in changelog.md with the change
• If PR includes UI updates/additions, its description has screenshots
• Behavior is as expected
• Clean, well-structured code

[codecov]

Codecov Report

Attention: Patch coverage is 2.94840% with 395 lines in your changes missing coverage. Please review.

Project coverage is 11%. Comparing base (f6e0da3) to head (4d28139).
Report is 37 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #1283    +/-   ##
========================================
- Coverage       16%     11%    -4%     
========================================
  Files          238     268    +30     
  Lines         7172    7303   +131     
========================================
- Hits          1078     799   -279     
- Misses        6094    6504   +410     

[thePeras]

✅ 🛫

[Process-ing]

I think that the code is overall quite good (props to using a class-orineted approach for the API) but I still have some recommendations and questions. I also noticed that if I login with federated authentication, and then logout (even after closing the app), the login page will try to open my browser without me selecting anything.

[Process-ing]

Nice job 🚀

[limwa]

Don't merge yet, there is an active discussion in slack

[DGoiana]

Great job here! Thanks for your effort, code is now much cleaner and well architectured 🚀