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

feat: add set_auth method #175

Merged
merged 4 commits into from
Aug 8, 2024
Merged

feat: add set_auth method #175

merged 4 commits into from
Aug 8, 2024

Conversation

grdsdev
Copy link
Contributor

@grdsdev grdsdev commented Aug 5, 2024

What kind of change does this PR introduce?

Feature

What is the current behavior?

Missing set_auth method for updating access token

What is the new behavior?

Add set_auth method which pushes a access_token event with new access token to all subscribed channels.

@grdsdev grdsdev marked this pull request as draft August 5, 2024 12:38
Copy link
Contributor

@juancarlospaco juancarlospaco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM so far.

@grdsdev grdsdev marked this pull request as ready for review August 7, 2024 11:13
Copy link
Contributor

@juancarlospaco juancarlospaco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@grdsdev grdsdev merged commit 5859c72 into master Aug 8, 2024
5 checks passed
@grdsdev grdsdev deleted the set_auth branch August 8, 2024 11:01
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