jbrowse-components icon indicating copy to clipboard operation
jbrowse-components copied to clipboard

Initialize @jbrowse/app-core to share code between desktop/web/embedded

Open cmdcolin opened this issue 3 years ago • 1 comments

This starts out a new package called @jbrowse/app-core. This first example adds "createInternetAccount/findInternetAccount" authentications to the root model

Xref https://github.com/mobxjs/mobx-state-tree/discussions/1842#discussioncomment-1972622

currently there is internet-account specific autorun that is created in the web/desktop root models instead of in app-core since it's hard to create extra tasks for afterCreate, but that could be remedied probably

cmdcolin avatar Sep 21 '22 16:09 cmdcolin

Could extension points be used for afterCreate?

On Wed, Sep 21, 2022 at 9:52 AM Colin Diesh @.***> wrote:

This starts out a new package called @jbrowse/app-core. This first example adds "createInternetAccount/findInternetAccount" authentications to the root model

Xref mobxjs/mobx-state-tree#1842 (comment) https://github.com/mobxjs/mobx-state-tree/discussions/1842#discussioncomment-1972622

currently there is internet-account specific autorun that is created in the web/desktop root models instead of in app-core since it's hard to create extra tasks for afterCreate, but that could be remedied probably

You can view, comment on, or merge this pull request online at:

https://github.com/GMOD/jbrowse-components/pull/3206 Commit Summary

File Changes

(10 files https://github.com/GMOD/jbrowse-components/pull/3206/files)

Patch Links:

  • https://github.com/GMOD/jbrowse-components/pull/3206.patch
  • https://github.com/GMOD/jbrowse-components/pull/3206.diff

— Reply to this email directly, view it on GitHub https://github.com/GMOD/jbrowse-components/pull/3206, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAASAFOKDMAKQIHTG5EXKHLV7M4NXANCNFSM6AAAAAAQSIKWHQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

rbuels avatar Sep 21 '22 17:09 rbuels

the alternative app core has landed in #3701 and #3661

cmdcolin avatar May 18 '23 18:05 cmdcolin