Skip to content

Conversation

@ArrayZoneYour
Copy link
Contributor

export default Model(stores) will raise error.

image

We don't know what returns from Model(...)

You will need to write 🤔

import Model from 'models/index'
const { useStore } = Model

So I revert the changes.

@ArrayZoneYour ArrayZoneYour self-assigned this Mar 27, 2019
@ArrayZoneYour ArrayZoneYour requested a review from leecade March 27, 2019 05:23
@codecov
Copy link

codecov bot commented Mar 27, 2019

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #88   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           4      4           
  Lines         206    206           
  Branches       29     29           
=====================================
  Hits          206    206

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce1fee9...cbc62d2. Read the comment docs.

Copy link
Contributor

@leecade leecade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try

export Model(stores)

// or

export {...Model(stores)}

// ignore, wrong synstx
export * from Model(stores)

@ArrayZoneYour ArrayZoneYour merged commit a17b1bc into master Mar 28, 2019
@ArrayZoneYour ArrayZoneYour deleted the docs/readme branch April 3, 2019 13:26
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.

4 participants