Skip to content
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

CIF-1907 - Build navigation component based on WCM Core Component #126

Merged
merged 4 commits into from
Apr 21, 2021

Conversation

LSantha
Copy link
Collaborator

@LSantha LSantha commented Apr 14, 2021

  • implemented Venia theme for navigation component v2

Related Issue

CIF-1907

How Has This Been Tested?

Manually.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes and the overall coverage did not decrease.
  • All unit tests pass on CircleCi.
  • I ran all tests locally and they pass.

@codecov
Copy link

codecov bot commented Apr 14, 2021

Codecov Report

Merging #126 (4063925) into dev (2c5b07b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                dev     #126   +/-   ##
=========================================
  Coverage     85.71%   85.71%           
  Complexity       18       18           
=========================================
  Files             1        1           
  Lines            28       28           
  Branches          4        4           
=========================================
  Hits             24       24           
  Misses            2        2           
  Partials          2        2           
Flag Coverage Δ Complexity Δ
unittests 85.71% <ø> (ø) 18.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 2c5b07b...4063925. Read the comment docs.

@LSantha LSantha added the enhancement New feature or request label Apr 15, 2021
@mhaack mhaack merged commit 6f8ca2f into dev Apr 21, 2021
@mhaack mhaack deleted the CIF-1907 branch April 21, 2021 09:55
@mhaack mhaack mentioned this pull request Apr 22, 2021
mhaack added a commit that referenced this pull request Apr 22, 2021
* @releng - update CIF Core Components dependency to SNAPSHOT version

* CIF-1718 - Adopt Magento GraphQL schema changes - Category UID

- update graphql dependency to 2.4.2 EE to align with CIF components

* CIF-2015 - Migrate to Azul Zulu JDK (#125)

* @trivial fix ui.frontend readme (#128)

* CIF-2039 - Missing root category field in catalog page properties (#127)

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * fixed root category field label in UI test

* CIF-1907 - Build navigation component based on WCM Core Component (#126)

* CIF-1907 - Build navigation component based on WCM Core Component

 * implemented Venia theme for navigation component v2

* CIF-1907 - Build navigation component based on WCM Core Component

 * fix test failures

Co-authored-by: Markus Haack <mhaack@adobe.com>

* @trivial - update dependencies (#129)

Co-authored-by: Levente Santha <levente@adobe.com>
Co-authored-by: Christophe Jelger <jelger@adobe.com>
Co-authored-by: Mark J. Becker <herzog31@users.noreply.github.com>
dplaton added a commit that referenced this pull request May 24, 2021
* @releng - update CIF Core Components dependency to SNAPSHOT version

* CIF-1718 - Adopt Magento GraphQL schema changes - Category UID

- update graphql dependency to 2.4.2 EE to align with CIF components

* CIF-2015 - Migrate to Azul Zulu JDK (#125)

* @trivial fix ui.frontend readme (#128)

* CIF-2039 - Missing root category field in catalog page properties (#127)

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * added UI test for catalog page properties extensions

* CIF-2039 - Missing root category field in catalog page properties

 * fixed root category field label in UI test

* CIF-1907 - Build navigation component based on WCM Core Component (#126)

* CIF-1907 - Build navigation component based on WCM Core Component

 * implemented Venia theme for navigation component v2

* CIF-1907 - Build navigation component based on WCM Core Component

 * fix test failures

Co-authored-by: Markus Haack <mhaack@adobe.com>

* @trivial - update dependencies (#129)

* CIF-1746 - Create CIF data types for Content Fragment Models (#102)

* enable content fragment models in Venia configuration

* CIF-2007 - Use latest releases of CIF Add-on for IT and UI tests (#132)

* CIF-2007 - Use latest release of CIF add-on from artifactory

* CIF-2007 - Use latest release of CIF add-on from artifactory

* CIF-2007 - Use latest release of CIF add-on from artifactory

* CIF-2007 - Use latest release of CIF add-on for AEM 6.5 from artifactory

* CIF-2007 - Fix UI tests

* @releng Update dependency versions (#134)

Update `AEM Core CIF Components` to 1.11.0 and `@adobe/aem-core-cif-react-components` to 1.11.0

Co-authored-by: Daniel Platon <platon@adobe.com>

* CIF-2086: Update react-components to be in sync with CIF Components version (#133)

* CIF-2086: Sync reac-components version on release

* CIF-2086: Commit updated npm files

* CIF-2086: Fix npm files commit profile

* CIF-2086: Update cif-react-components dependency

* Remove package-lock.json from .gitignore

Co-authored-by: Daniel Platon <platon@adobe.com>

Co-authored-by: Levente Santha <levente@adobe.com>
Co-authored-by: Christophe Jelger <jelger@adobe.com>
Co-authored-by: Mark J. Becker <herzog31@users.noreply.github.com>
Co-authored-by: Markus Haack <mhaack@adobe.com>
Co-authored-by: Daniel Platon <platon@adobe.com>
Co-authored-by: Laurentiu Magureanu <magurean@adobe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants