-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fix life cycle dependency appstore in api part 01 #960
Fix life cycle dependency appstore in api part 01 #960
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you explain a bit more your changes and the plans how you would like to continue after this step? Some changes seem not that intuitive to me. :)
We do agree that in the end we can say it is fine that the stores
depend on api
, but not the other way around?
I apologize that it took me too long to reply to your comments.
Please let me know if I do bad code, I will fix it. |
Our global
AppStore
andwebApi
are used everywhere as desired, also inAccountApi
,AssetsApi
,ChainApi
andCodecApi
. This PR #525 (Fix cyclic dependency between store <> api) is the first step in solving the problem.AppStore
orwebApi
are no longer used inAccountApi
,AssetsApi
,ChainApi
.Yes some functions were moved into
Api
but this is temporarily. After merging this we will removeAppStore
fromEncointerApi
and then fromApi
.