Skip to content

Conversation

@MickaelCa
Copy link

@MickaelCa MickaelCa commented Sep 8, 2023

we use Google SSO so I added it

@MickaelCa MickaelCa marked this pull request as ready for review September 8, 2023 17:29
@MickaelCa MickaelCa force-pushed the feat/add-google-auth branch from 9ab5208 to 100853f Compare January 10, 2024 09:18
@MickaelCa
Copy link
Author

MickaelCa commented Jan 10, 2024

PR rebased for conflicts

@MickaelCa MickaelCa force-pushed the feat/add-google-auth branch from 100853f to f05789f Compare March 11, 2024 00:21
@MickaelCa
Copy link
Author

any chance to see this merged ?

@TheoAlloin
Copy link

@sztwiorok any chance to review this PR?

@sztwiorok
Copy link
Contributor

@MickaelCa @TheoAlloin We will review and merge it in a week

@sztwiorok
Copy link
Contributor

we checked the code and everything is fine. @MickaelCa could you make some small change in the code to run the GHA tests again?

@MickaelCa MickaelCa force-pushed the feat/add-google-auth branch from f05789f to 6959927 Compare October 10, 2024 07:37
@MickaelCa
Copy link
Author

@sztwiorok I rebased on main and force-pushed, should be enough!

@sztwiorok
Copy link
Contributor

@MickaelCa thanks
code style tests failed, could you look at this?

@sztwiorok
Copy link
Contributor

@MickaelCa, there are problems with unit tests, could you look at this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants