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

Fixes bug #110 #111

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Fixes bug #110 #111

merged 1 commit into from
Jul 31, 2024

Conversation

MaxFrax
Copy link
Contributor

@MaxFrax MaxFrax commented Sep 10, 2023

State becomes optional in AuthResult.from_dict.

Fixes bug #110

@LukasKaspras
Copy link

@MaxFrax has there been any feedback yet?

I also found a bug (incl. a fix) - #143 - but if community PR's are not taken into consideration, I don't want to invest time into creating my own PR.

@MaxFrax
Copy link
Contributor Author

MaxFrax commented Jul 29, 2024

@LukasKaspras Unfortunately, I have never received any feedback.

@lefcha lefcha self-assigned this Jul 31, 2024
@lefcha lefcha self-requested a review July 31, 2024 12:15
@lefcha lefcha merged commit 92cb83f into Doist:main Jul 31, 2024
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