Skip to content

Conversation

@mfranzke
Copy link
Contributor

@mfranzke mfranzke commented Nov 13, 2021

Summary of changes:

Updated our redux version, as redux-thunk introduced redux 4 as a peerDependency with a minor update.

The breaking changes have been collected in this issue: reduxjs/redux#1342
None of these are conflicting with our implementation.

This would solve errors and warnings like the following:

npm WARN [email protected] requires a peer of redux@^4 but none is installed. You must install peer dependencies yourself.

mfranzke and others added 30 commits April 24, 2020 16:56
missed to replace mustache initially and incorrect handlebars two times (both in package name and URL)
This reverts commit 7812878, reversing
changes made to 26db979.
This reverts commit 26db979, reversing
changes made to fc90750.
@mfranzke mfranzke marked this pull request as ready for review November 13, 2021 16:07
@mfranzke mfranzke requested a review from sghoweri as a code owner November 13, 2021 16:07
@JosefBredereck JosefBredereck merged commit a75ee0d into pattern-lab:dev Nov 21, 2021
@mfranzke mfranzke deleted the refactor/updated-redux branch November 21, 2021 15:10
antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
* chore(docs): some typos

missed to replace mustache initially and incorrect handlebars two times (both in package name and URL)

* Revert "Merge pull request pattern-lab#2 from pattern-lab/dev"

This reverts commit 7812878, reversing
changes made to 26db979.

* Revert "Merge pull request pattern-lab#1 from pattern-lab/dev"

This reverts commit 26db979, reversing
changes made to fc90750.

* Revert "chore(docs): some typos"

This reverts commit 271dc8d.

* refactor: updated our redux version, as redux-thunk introduced redux 4 as a peerDependency with a minor update

* Revert "Merge branch 'dev' into refactor/updated-redux"

This reverts commit 86baa4c, reversing
changes made to aa64967.

Co-authored-by: Maximilian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants