Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(oauth): github and discord login error #323

Merged
merged 5 commits into from
Nov 11, 2023

Conversation

zz5840
Copy link
Contributor

@zz5840 zz5840 commented Nov 10, 2023

fixed #322, fixed #302, close #325

Error in Microsoft (#322) still need to be located.

@zz5840
Copy link
Contributor Author

zz5840 commented Nov 10, 2023

@stonith404 Could you please build a dev image and let them test it?

@stonith404
Copy link
Owner

Great thank you! The dev image is pushed.

@stonith404
Copy link
Owner

Great, I'll just wait until we get feedback from the users if it works. Then I'll merge it :)

@zz5840
Copy link
Contributor Author

zz5840 commented Nov 10, 2023

@stonith404 Can you update the dev image to make changes in 4af97e4 take effect? It's important for locating the error in Microsoft authentication.

@stonith404
Copy link
Owner

@zz5840 👍 Pushed

@zz5840
Copy link
Contributor Author

zz5840 commented Nov 11, 2023

@stonith404 Maybe they are rest today, I think we should merge this pr and release a new version to fix discord and github first. I'll open a new pr to fix error in Microsoft.

@stonith404
Copy link
Owner

@zz5840 Makes sense. Thank you for the PR!

@stonith404 stonith404 merged commit fd44f42 into stonith404:main Nov 11, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants