-
Notifications
You must be signed in to change notification settings - Fork 40
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
Default Header Locale Formatter #520
Conversation
… type as we are doing it now.
Codecov Report
@@ Coverage Diff @@
## dev-v3.0.1 #520 +/- ##
==============================================
+ Coverage 87.69% 87.75% +0.05%
==============================================
Files 92 93 +1
Lines 4112 4138 +26
==============================================
+ Hits 3606 3631 +25
- Misses 506 507 +1 |
/// | ||
public struct DefaultHeadersFormatter { | ||
|
||
typealias httpHeaderConstants = HttpConnectionConstants.Header |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we still need this line?
/// - Parameter unformattedLocale: The unformatted locale as a string | ||
/// - Returns: The formatted locale as a string or nil if not able to format it. | ||
/// | ||
public static func getFormattedLocale(unformattedLocale: String) -> String { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_ unformattedLocale
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Decided to change the name as well. Lmk what you think.
In v5 we format the locale. Here we format the locale as in v5 except we do it in the
NetworkRequest
to keep it centrally located and not rely on each extension to do it.