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

feat(taxonomies): Adding variant for the description list, file and card components - front-2012 #597

Merged
merged 5 commits into from
Feb 11, 2021

Conversation

planctus
Copy link
Contributor

PR description

Please drop a few lines about the PR: what it does, how to test it, etc.

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

@planctus planctus added the pr: wip Mark the PR as WIP label Feb 11, 2021
@planctus planctus added pr: review needed Use this label to show that your PR needs to be review tag: new feature and removed pr: wip Mark the PR as WIP labels Feb 11, 2021
@planctus planctus marked this pull request as ready for review February 11, 2021 14:12
@emeryro emeryro self-assigned this Feb 11, 2021
@emeryro emeryro added pr: under review and removed pr: review needed Use this label to show that your PR needs to be review labels Feb 11, 2021
@@ -0,0 +1,64 @@
import adapter from '../adapter';

const specsTileTaxonomy = {
Copy link
Contributor

Choose a reason for hiding this comment

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

can't you import the specs file instead of duplicating it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I forgot that one, that's how i did it before the specs were available, sure!! :)

Copy link
Contributor

@emeryro emeryro left a comment

Choose a reason for hiding this comment

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

all good

@emeryro emeryro merged commit 56efa38 into develop Feb 11, 2021
@emeryro emeryro deleted the front-2012 branch February 11, 2021 14:58
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.

2 participants