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

Add Basic Auth. #99

Merged
merged 1 commit into from
Jun 15, 2022
Merged

Add Basic Auth. #99

merged 1 commit into from
Jun 15, 2022

Conversation

bastienuh
Copy link
Contributor

See #97.

Not sure I made it well, and not sure if I have to merge with master.
I made my change from last commit of tag v2.5.0.
I wanted to make a PR on "tag" but obviously not possible...

@Jeroen-G
Copy link
Owner

I have created the branch 2.6.0 for you to target instead of master!

@bastienuh bastienuh changed the base branch from master to 2.6.0 June 15, 2022 10:20
@bastienuh
Copy link
Contributor Author

bastienuh commented Jun 15, 2022

Perfect, thanks, I changed the commit.

@bastienuh
Copy link
Contributor Author

Oh.... I took the tag 2.5.0, and not the 2.5.1.
Do you want me close this PR and make another one with the correct commit? Or maybe you can merge everything in the 2.6.0 branch?

Sorry !

@Jeroen-G
Copy link
Owner

There's no problem as far as I can see!

@bastienuh
Copy link
Contributor Author

Perfect, so can you merge and close ? 😇

@Jeroen-G Jeroen-G merged commit 84b7fbb into Jeroen-G:2.6.0 Jun 15, 2022
@Jeroen-G
Copy link
Owner

Merged, maybe you can test it with requiring the dev-2.6.0 requirement.

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