Skip to content

🐛 amp-story-shopping does not display CTA if all tags are invalid and Fixes remote shopping data loading on refresh with page attachment open #38035

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.

Sign up for GitHub

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

Merged
jshamble merged 28 commits into from
Apr 19, 2022
Merged

🐛 amp-story-shopping does not display CTA if all tags are invalid and Fixes remote shopping data loading on refresh with page attachment open #38035

Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
c32eeaa
added fixed for shopping tag rendering
jshamble Apr 7, 2022
9c807f0
added shopping json patch that fixed regression
jshamble Apr 11, 2022
86dd0f3
resolved promise .all
jshamble Apr 11, 2022
ab39a72
updated template
jshamble Apr 11, 2022
0bb58ce
refactored state update
jshamble Apr 11, 2022
52c0312
fixed promise resolution
jshamble Apr 12, 2022
2088558
added more descriptive error message
jshamble Apr 15, 2022
5618cb7
cahnged order of ternary
jshamble Apr 15, 2022
05cac5f
added history check for page refresh
jshamble Apr 18, 2022
d341d6b
updated hsitory service to store all data, not jsut the activeData
jshamble Apr 18, 2022
781fe65
added shopping element
jshamble Apr 18, 2022
2103d85
added shopping attachment logic for page refresh, added back active p…
jshamble Apr 18, 2022
0b685ac
added set history state test
jshamble Apr 18, 2022
8d05b46
cleaned up some minor nits
jshamble Apr 18, 2022
a2e94fe
added unit test for shopping tag history
jshamble Apr 18, 2022
f818401
history stub
jshamble Apr 18, 2022
d0f5d36
added shopping if clause optimizations
jshamble Apr 19, 2022
a87a927
added dependencies
jshamble Apr 19, 2022
cfb87f2
added dependencies
jshamble Apr 19, 2022
0b78d51
added dependencies
jshamble Apr 19, 2022
8a3fcd3
updated unit tests
jshamble Apr 19, 2022
723981b
updated unit tests
jshamble Apr 19, 2022
e05b6ce
updated unit tests
jshamble Apr 19, 2022
ed8d8f3
removed check
jshamble Apr 19, 2022
b75b6e5
browser test safari
jshamble Apr 19, 2022
b44c364
browser test safari
jshamble Apr 19, 2022
b5130a5
added better url checks
jshamble Apr 19, 2022
66e7b9a
added stuff
jshamble Apr 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
added dependencies
  • Loading branch information
@jshamble
jshamble committed Apr 19, 2022
commit 0b78d5197b2cbc90142191ad5dceb5f7554d2257
2 changes: 1 addition & 1 deletion build-system/test-configs/dep-check-config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ exports.rules = [
'extensions/amp-story-shopping/0.1/amp-story-shopping-attachment.js->extensions/amp-story/1.0/variable-service.js',
'extensions/amp-story-shopping/0.1/amp-story-shopping-attachment.js->extensions/amp-story/1.0/story-analytics.js',
'extensions/amp-story-shopping/0.1/amp-story-shopping-attachment.js->extensions/amp-story/1.0/history.js',
'extensions/amp-story-shopping/0.1/amp-story-shopping-tag.js->extensions/amp-story/1.0/variable-servicde.js',
'extensions/amp-story-shopping/0.1/amp-story-shopping-tag.js->extensions/amp-story/1.0/variable-service.js',
'extensions/amp-story-shopping/0.1/amp-story-shopping-tag.js->extensions/amp-story/1.0/story-analytics.js',
'extensions/amp-story-shopping/0.1/amp-story-shopping-tag.js->extensions/amp-story/1.0/history.js',

Expand Down