Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Shopify analytics 2 #1325
Shopify analytics 2 #1325
Changes from 63 commits
5ac8f88
4e4f357
549fe32
7f15d0f
da764aa
2f843ff
1a09d08
be26143
d584497
bb3fa18
9de0de1
8bb20d2
bdf1f0b
8314820
f31490f
29a8d15
e7864bf
e68e37a
5b05304
5268b12
e8bb405
eefc90f
bd79723
1a257ca
8e1f80d
f80f0aa
c5d02c6
82dc942
da18ce1
7ddeac3
ddd7314
3e4b46c
af6e2d1
32a6d58
d65d8c2
12b0da5
2b8a6d2
490e41d
3bd1d5d
a61d87c
a9ba5ec
cdd13d5
33621b5
d087fe5
d6105c6
87bab2e
829b27b
e37bc0b
9f56514
b966253
ad9eb06
5b9e1d3
a7f1aac
906bdd6
476a6c1
7921d1d
4b02072
0674fdd
9c3b234
817de86
5ae8b85
c7ea791
84ea431
c142b58
f6a4da4
c7218df
a4d2fbc
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Would we want to expose other cookie settings?
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.
No
_shopify_y
and 30 mins for_shopify_s
/
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.
This might change when CHIP is fully supported
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.
Why make this change? The request object itself has both the url and headers already on it. And it's more flexible.
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.
The
request
object is garbage collected after we return a response. This was fine when there was only 1 server analytics connector. When there are more than 1 server analytics connector, we get into situation where the request no longer exist for the multiple server analytics connectors to process