Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: Migrating From Redux #108

Merged
merged 11 commits into from
Apr 24, 2020
Merged

Docs: Migrating From Redux #108

merged 11 commits into from
Apr 24, 2020

Conversation

alvinhui
Copy link
Member

@alvinhui alvinhui commented Apr 23, 2020

  • Docs & Examples
  • Export context, use: store.context

@alvinhui alvinhui self-assigned this Apr 23, 2020
@codecov-io
Copy link

codecov-io commented Apr 23, 2020

Codecov Report

Merging #108 into release/1.4.0 will increase coverage by 0.23%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                Coverage Diff                @@
##           release/1.4.0     #108      +/-   ##
=================================================
+ Coverage          87.91%   88.15%   +0.23%     
=================================================
  Files                 20       20              
  Lines                546      557      +11     
  Branches             121      123       +2     
=================================================
+ Hits                 480      491      +11     
  Misses                56       56              
  Partials              10       10              
Impacted Files Coverage Δ
src/plugins/provider.tsx 100.00% <100.00%> (ø)
src/plugins/effects.ts 96.42% <0.00%> (+2.31%) ⬆️

@chenbin92 chenbin92 merged commit 30746b6 into release/1.4.0 Apr 24, 2020
@delete-merged-branch delete-merged-branch bot deleted the docs/migration branch April 24, 2020 06:01
@alvinhui alvinhui mentioned this pull request Apr 24, 2020
alvinhui added a commit that referenced this pull request Apr 24, 2020
* chore: new version

* fix: add meta type in ModelEffects (#102)

* Add @ice/store Unit Test (#84)

* chore: add dependency

* test: add helper and init test

* test: add createStore test

* test: add withModel withModelActions withModelEffectsState test

* test: add throw error when getting unexist model

* chore: new version

* test: add global actions test

* test: add createModel test

* test: add createUseContainer test

* chore: createModel

* test: add createContainer test

* test: update test cases

* test: add useModel test

* test: add class component test

* test: update class component test

* test: add createStore options test

* test: add appendReducer test

* test: add appendReducer test

* chore: lint

* test: add utils test

* docs: add badge

* chore: remove unnecessary comment

* test: update converter test

* feat: add comment to pr

* fix: fix by the comments

* chore: update codecov config

* test: update test

* test: update converter test case

* test: update examples

* test: update todos example

* chore: update modelEffects type

Co-authored-by: alvinhui <alvin.hui@qq.com>

* fix: effects should return an object (#105)

* fix: effects should return an object

* chore: lint

* Better example (#106)

* chore: add classComponent example

* chore: rename async function

* chore: visibility logic

* docs: add examples section

* feat: withModel (#104)

* feat: withModel init

* refactor: model apis

* feat: model for withModel

* feat: ts support for withModel

* chore: typo

* chore: lint

* test: disable loading and error

* test: mock value

* chore: playload && meta is option

* chore: typo

* docs: withModel

* refactor: class and function component demo

* chore: class component support

* chore: remove console

* chore: lint

* chore: withModel example

* chore: remove product

* chore: undo

* chore: typo

* Docs: Migrating From Redux  (#108)

* chore: example @ice/store version

Co-authored-by: Hengchang Lu <44047106+luhc228@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants