Skip to content
This repository has been archived by the owner on May 2, 2023. It is now read-only.

refactor(demo): Site Header Standardised, mandatory/optional knobs - FRONT-919 #384

Merged
merged 5 commits into from Mar 24, 2020

Conversation

planctus
Copy link
Contributor

PR description

WARNING

There is an update in the index.js of the story-utils package

QA Checklist

In order to ensure a safe and quick review, please check that your PR follow those guidelines:

  • I have put the vanilla component as devDependencies
  • I have put the specs package as devDependencies
  • I have added the components directly used in the twig file (with include or embed) as dependencies
  • My component is listed in @ecl-twig/ec-components's dependencies
  • My variables n`aming follow the guidelines (snake case for twig)
  • I have provided tests
  • I have provided documentation (for the "notes" tab)
  • If my local yarn.lock contains changes, I have committed it
  • I have given my PR the proper label (pr: review needed to indicate that I'm done and now waiting for a review ,pr: wip to indicate that I'm actively working on it ...)

@planctus planctus added pr: review needed Use this label to show that your PR needs to be review tag: enhancement labels Mar 23, 2020
@papegaill papegaill added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Mar 24, 2020
@papegaill papegaill merged commit 556845e into develop Mar 24, 2020
@papegaill papegaill deleted the FRONT-919 branch March 24, 2020 15:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants