Skip to content

fix: effects should return an object#105

Merged
alvinhui merged 2 commits into
release/1.4.0from
fix/effects
Apr 23, 2020
Merged

fix: effects should return an object#105
alvinhui merged 2 commits into
release/1.4.0from
fix/effects

Conversation

@alvinhui

Copy link
Copy Markdown
Member

No description provided.

@alvinhui alvinhui requested review from chenbin92 and luhc228 April 21, 2020 06:45
@alvinhui alvinhui self-assigned this Apr 21, 2020
@codecov-io

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@                Coverage Diff                @@
##           release/1.4.0     #105      +/-   ##
=================================================
+ Coverage          87.91%   87.93%   +0.02%     
=================================================
  Files                 20       20              
  Lines                546      547       +1     
  Branches             121      121              
=================================================
+ Hits                 480      481       +1     
  Misses                56       56              
  Partials              10       10              
Impacted Files Coverage Δ
src/plugins/effects.ts 94.44% <100.00%> (+0.32%) ⬆️

@alvinhui alvinhui merged commit 6a514af into release/1.4.0 Apr 23, 2020
@delete-merged-branch delete-merged-branch Bot deleted the fix/effects branch April 23, 2020 04:31
This was referenced Apr 23, 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