Skip to content

[RTC-216/217] Allow token to be set in config and update docs#5

Merged
LVala merged 7 commits intomasterfrom
set-token-in-config
Apr 13, 2023
Merged

[RTC-216/217] Allow token to be set in config and update docs#5
LVala merged 7 commits intomasterfrom
set-token-in-config

Conversation

@LVala
Copy link
Copy Markdown
Collaborator

@LVala LVala commented Apr 12, 2023

This PR contains RTC-126 and RTC-217 as they are closely related.

@LVala LVala self-assigned this Apr 12, 2023
@LVala LVala requested review from bblaszkow06 and mickel8 April 12, 2023 15:53
Comment thread lib/jellyfish/client.ex Outdated
@LVala LVala requested a review from mickel8 April 13, 2023 08:04
Comment thread README.md Outdated
Comment thread lib/jellyfish/client.ex
def new(address, token), do: build_client(address, token)

@doc """
Creates new instance of `t:Jellyfish.Client.t/0`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

JFYI, you can also write t:t/0 but the result will be slighlity different

Comment thread lib/jellyfish/room.ex Outdated
@LVala LVala requested a review from mickel8 April 13, 2023 09:23
@LVala LVala merged commit 30492c8 into master Apr 13, 2023
@LVala LVala deleted the set-token-in-config branch April 13, 2023 09:34
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.

2 participants