Skip to content

Conversation

cjelger
Copy link
Contributor

@cjelger cjelger commented May 12, 2020

…ponent

  • add proxy product component for the library
  • add a page component that adds the store-view and graphql endpoint for the client-side code
  • add JSON mock response and images for the product component
  • product images are simply stored in the DAM (see 2nd screenshot): I left all the renditions as done for the WCM components library assets

Screenshot 2020-05-12 at 11 47 49

Screenshot 2020-05-12 at 11 49 55

…ponent

- add proxy product component for the library
- add a `page` component that adds the store-view and graphql endpoint for the client-side code
- add JSON mock response and images for the product component
@cjelger cjelger added the documentation Documentation issue label May 12, 2020
@codecov
Copy link

codecov bot commented May 13, 2020

Codecov Report

Merging #267 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #267   +/-   ##
=========================================
  Coverage     63.20%   63.20%           
  Complexity      754      754           
=========================================
  Files           173      173           
  Lines          5354     5354           
  Branches        840      840           
=========================================
  Hits           3384     3384           
  Misses         1855     1855           
  Partials        115      115           
Flag Coverage Δ Complexity Δ
#jest 40.82% <ø> (ø) 0.00 <ø> (ø)
#karma 94.88% <ø> (ø) 0.00 <ø> (ø)
#unittests 85.20% <ø> (ø) 754.00 <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e79f7f...78df711. Read the comment docs.

@laurentiumagureanu laurentiumagureanu merged commit 3facf32 into master May 13, 2020
@laurentiumagureanu laurentiumagureanu deleted the CIF-1382 branch May 13, 2020 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation issue verified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants