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

refactor(knobs): timeline knobs and labels - FRONT-923 #386

Merged
merged 10 commits into from Mar 25, 2020
Merged

Conversation

Joosthe
Copy link
Contributor

@Joosthe Joosthe commented Mar 24, 2020

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

@Joosthe Joosthe added pr: wip Mark the PR as WIP tag: enhancement labels Mar 24, 2020
@Joosthe Joosthe added dependencies Pull requests that update a dependency file and removed pr: wip Mark the PR as WIP labels Mar 24, 2020
@Joosthe Joosthe changed the title refactor(knobs): progression commit - FRONT-923 refactor(knobs): timeline knobs and labels - FRONT-923 Mar 24, 2020
@papegaill papegaill added pr: under review and removed dependencies Pull requests that update a dependency file labels Mar 24, 2020
@Joosthe Joosthe added pr: review needed Use this label to show that your PR needs to be review and removed pr: modification needed labels Mar 25, 2020
@Joosthe Joosthe merged commit 387ef86 into develop Mar 25, 2020
@Joosthe Joosthe deleted the FRONT-923 branch March 25, 2020 12:37
@planctus planctus removed the pr: review needed Use this label to show that your PR needs to be review label Mar 25, 2020
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