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(s3): allow custom endpoint #25

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

nipeharefa
Copy link

@nipeharefa nipeharefa commented Oct 6, 2021

Hi @brunopadz
this is my first PR and just fix issue #17 to allow custom endpoint

Copy link
Contributor

@brunopadz brunopadz left a comment

Choose a reason for hiding this comment

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

Please check the comment, I believe it's a typo, and also update the docs with the new feature.

Edit: Rebase with latest changes from develop.

action.yml Outdated Show resolved Hide resolved
@nipeharefa nipeharefa force-pushed the test/custome-endpoint branch 2 times, most recently from 324720b to 09b19e1 Compare October 11, 2021 10:51
@brunopadz
Copy link
Contributor

@nipeharefa could you please update the docs with some examples?

cheers!

@nipeharefa
Copy link
Author

Hello, @brunopadz just update the docs with some example with custom endpoint

@brunopadz
Copy link
Contributor

LGTM 👍

@amendoa could you please take a look into it?

brunopadz
brunopadz previously approved these changes Nov 2, 2021
@Sytten
Copy link

Sytten commented Jul 25, 2022

Anything else left before this can be merged?

@brunopadz
Copy link
Contributor

@Sytten I'm not a mantainer of this project anymore. You have to check with the new mantainers.

Otherwise, feel free to fork the project and apply the changes. ;)

@saniales
Copy link

@amendoa how is the review going for this? I think it could bring a very good impact

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants