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

Allow setting Accept-Language header #483

Merged
merged 2 commits into from
Oct 25, 2021
Merged

Allow setting Accept-Language header #483

merged 2 commits into from
Oct 25, 2021

Conversation

vacom13
Copy link
Contributor

@vacom13 vacom13 commented Oct 24, 2021

I have given like an option whether the user wants to send it as an accept language header. However I am not sure if this is what you had in mind when talking about this feature. So do tell me if I should instead do something else.
P.S Sorry for the delay, just fell ill.
Implements #445

@benbusby
Copy link
Owner

Thanks @vacom13! No need to apologize for any delays, there's no impending deadline for the next release or anything. I'm just happy you're helping out!

I'll try to review later today or possibly tomorrow. I'm in the middle of moving, so my time for this stuff is a bit sporadic.

@vacom13
Copy link
Contributor Author

vacom13 commented Oct 24, 2021

@benbusby oh great. Happy to help!

@benbusby benbusby changed the title Implements #445 Allow setting Accept-Language header Oct 25, 2021
@benbusby benbusby linked an issue Oct 25, 2021 that may be closed by this pull request
Copy link
Owner

@benbusby benbusby left a comment

Choose a reason for hiding this comment

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

Looks good, thanks @vacom13!

@benbusby benbusby merged commit 5118ddb into benbusby:main Oct 25, 2021
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.

[FEATURE] Allow set Accept-Language
2 participants