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

feat: EC Gallery - INNO-1384 #76

Merged
merged 10 commits into from
Mar 29, 2019
Merged

Conversation

emeryro
Copy link
Contributor

@emeryro emeryro commented Mar 13, 2019

PR description

Add component gallery (EC only)

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 naming 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 ...)

@emeryro emeryro self-assigned this Mar 13, 2019
@emeryro emeryro added pr: wip Mark the PR as WIP and removed pr: on hold labels Mar 22, 2019
@emeryro emeryro added pr: review needed Use this label to show that your PR needs to be review and removed pr: wip Mark the PR as WIP labels Mar 22, 2019
@emeryro emeryro removed their assignment Mar 22, 2019
@yhuard yhuard self-assigned this Mar 22, 2019
@yhuard yhuard added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Mar 22, 2019
yhuard
yhuard previously requested changes Mar 22, 2019
Copy link
Contributor

@yhuard yhuard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small fixes needed :)

@@ -0,0 +1,3 @@
__snapshots__
*.story.js
*.test.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can ignore demo too

expect.assertions(1);

const withExtraClasses = merge(demoData, {
extra_classes: 'custom-link custom-link--test',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

custom-class? :)

"@ecl-twig/ec-component-icon": "^0.0.1-alpha",
"@ecl-twig/ec-component-link": "^0.0.1-alpha"
},
"devDependencies": {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the order is not right (should be alphabetical)

@yhuard yhuard removed their assignment Mar 22, 2019
@emeryro emeryro added pr: review needed Use this label to show that your PR needs to be review and removed pr: modification needed labels Mar 26, 2019
@kalinchernev kalinchernev self-assigned this Mar 29, 2019
@kalinchernev kalinchernev added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Mar 29, 2019
@kalinchernev kalinchernev dismissed yhuard’s stale review March 29, 2019 13:45

Requested changes have been applied.

@kalinchernev kalinchernev merged commit 5752dc6 into master Mar 29, 2019
@kalinchernev kalinchernev deleted the ec-component-gallery-INNO-1384 branch March 29, 2019 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants