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: active storage: make non-aws users happy #355

Merged
merged 1 commit into from
Aug 1, 2022

Conversation

namehorn
Copy link
Contributor

@namehorn namehorn commented Jul 29, 2022

@jdenquin
Copy link
Contributor

jdenquin commented Aug 1, 2022

Hello @namehorn,

Thanks for your PR!
It's definitely a nice adding to support some other S3!

I merge this one and we will add the support of the new env var into https://github.com/getlago/lago/blob/main/docker-compose.yml and in our documentation.

Cheers,

@jdenquin jdenquin merged commit 7546cb3 into getlago:main Aug 1, 2022
@namehorn
Copy link
Contributor Author

namehorn commented Aug 1, 2022

Hi @jdenquin,

are you down for some more merges regarding native GCS support & helm kubernetes integration?

I have a few things lying around here.

@jdenquin
Copy link
Contributor

jdenquin commented Aug 1, 2022

@namehorn can you join our slack community? getlago.com/slack
We can discuss those subjects there!

But yes you're welcome to create some other PR around this!

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