Skip to content

feat: Refactor OAuth management to manage Logout#22

Merged
taorepoara merged 6 commits intomainfrom
logout
Feb 5, 2025
Merged

feat: Refactor OAuth management to manage Logout#22
taorepoara merged 6 commits intomainfrom
logout

Conversation

@taorepoara
Copy link
Member

@taorepoara taorepoara commented Feb 5, 2025

About this PR

OAuth2 management with a controller. This ease custom implementation for the OAuth2.

Technical highlight/advice

Some elements are keep but deprecated in order to migrate to the controller system at the next breaking release.

How to test my changes

In a LenraApplication sub widget, you can get the LenraApp instance and logout with the next code:

LenraApp.of(context).oauth2Controller.logout();

Checklist

  • I didn't over-scope my PR
  • My PR title matches the commit convention
  • I did not include breaking changes
  • I made my own code-review before requesting one

I included unit tests that cover my changes

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

I added/updated the documentation about my changes

  • 📜 README.md
  • 📕 docs/*.md
  • 📓 docs.lenra.io
  • 🙅 no documentation needed

@taorepoara taorepoara self-assigned this Feb 5, 2025
@taorepoara taorepoara marked this pull request as ready for review February 5, 2025 14:55
@taorepoara taorepoara merged commit 5b8262d into main Feb 5, 2025
1 check passed
@taorepoara taorepoara deleted the logout branch February 5, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant