search

flyntwp / flynt-starter-theme

The starter theme for building Flynt projects.
MIT License
69 stars 4 forks source link

BaseComponent: BlockCookieNotification #230

Closed bdbch closed 7 years ago

dgrdl commented 7 years ago

@bdbch please always make sure to send your PRs to the develop branch, not master. (master is the latest release version of the theme)

EDIT: of course also make sure you start developing from develop ;)

dgrdl commented 7 years ago

@bdbch I just saw that the yarn.lock file didn't change, but the package.json did. Please always use yarn to install your dependencies.

bdbch commented 7 years ago

@Qakulukiam are there still open changes to be made? I updated this pull request again.

dgrdl commented 7 years ago

Will be moved to a new component repository.