-
Notifications
You must be signed in to change notification settings - Fork 137
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
Prepare for 4.0.0 #634
Prepare for 4.0.0 #634
Conversation
MobileBuy/sample/src/shopify/java/com/shopify/sample/SampleApplication.java
Outdated
Show resolved
Hide resolved
1b2d08a
to
90f9c84
Compare
8a10c9e
to
006d1a4
Compare
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.
Still looks good, just some non-blocking questions
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.
Thanks, @jmignac. Left a few suggestions for you
006d1a4
to
8862e2b
Compare
e91f6c4
to
f880cdd
Compare
val httpCache = httpCacheConfig.let { config -> | ||
when (config) { | ||
is HttpCacheConfig.DiskLru -> { | ||
val version = BuildConfig.VERSION_NAME | ||
val tmp = (endpointUrl.toString() + "/" + version + "/" + accessToken).toByteArray(Charset.forName("UTF-8")) | ||
val tmp = (endpointUrl.toString() + "/" + version + "/" + accessToken + "/" + locale).toByteArray(Charset.forName("UTF-8")) |
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.
@sav007 is this the only change needed to make caching work with different locales?
f880cdd
to
25be3cc
Compare
25be3cc
to
c942067
Compare
Release notes
GraphClient.build()
now can takelocale
as an extra param, that sets theAccept-Language
header for retrieving translated values from Storefront API. Documentation updatedFKP
,GIP
andSHP
currenciespresentmentUnitPrices
,unitPrice
andunitPriceMeasurement
onProductVariant
media
field onProduct
: