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

Get server from environment variable #15

Closed
wants to merge 2 commits into from
Closed

Get server from environment variable #15

wants to merge 2 commits into from

Conversation

Massolari
Copy link

Currently, the server is fixed on https://matrix.org.
This PR adds support for custom server by reading it from the environment variable MATRIX_SERVER.

Besides from that, to make the development easier, I added the dotenv package that read the environment variables from a .env file and added a .env.example file so we have a template of which variables the application reads

@Massolari Massolari changed the title Feat: get server from env Get server from environment variable Apr 26, 2024
@smolck
Copy link
Owner

smolck commented Apr 26, 2024

Looks good, I might read a little into the dotenv crate before merging though. Also needs a cargo fmt to pass CI.

@smolck smolck added the enhancement New feature or request label Apr 26, 2024
@Massolari Massolari closed this by deleting the head repository Apr 29, 2024
@smolck
Copy link
Owner

smolck commented May 2, 2024

Added in 37166ac, also I realized because of the way the bot works you can already use it in other servers, forgot about that it's been a minute since I touched this thing haha. Anyways I added more info to the README for that.

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.

2 participants