-
Notifications
You must be signed in to change notification settings - Fork 14
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
Feature: add custom domain support #167
Conversation
9b77354
to
1fe4a09
Compare
efc265a
to
30e19e8
Compare
Assigned BLACK, as it exceeds Llama context size |
ee808e8
to
ad84357
Compare
CI/CD fails because of an oversight on my side... There's two different ways messages are being returned from the API.
The recent merge of aleph-im/aleph-sdk-python#81 now pulls messages from endpoint 2) and therefore validation of the This PR aleph-im/aleph-message#40 should fix it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, only necessary change is to replace @coro
annotations by using app = AsyncTyper()
from aleph_client.utils
7539a33
to
d2a5c25
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@hoh
to review/merge after =>
aleph-im/aleph-sdk-python#55