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: support v4.0.0-alpha1 #7

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

ooliver1
Copy link
Member

@ooliver1 ooliver1 commented Apr 25, 2024

This update refactors thang to support the 'new' users update.

This includes:

  • using disguise for setting name and avatar
  • ignoring own messages via author id instead of Bridge- prefix
  • refactoring redis usage for 0.25.3 where async connections are now multiplexed so have new method names and types, as well as Client::get_async_pubsub
  • removing multiple Eludris URL support soon to be in favour of multiple real channels

redis now uses multiplexed connections (aio::Connection deprecated)
this requires refactoring types as well as how `pubsub` is obtained (directly from client)
this also requires the redis refactor from the previous commit
this also requires the redis refactor from 2 commits ago
@ooliver1 ooliver1 added the enhancement New feature or request label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant