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

Maybe: unify the two pointer()s and move into app #128

Open
issa-tseng opened this issue Aug 28, 2018 · 0 comments
Open

Maybe: unify the two pointer()s and move into app #128

issa-tseng opened this issue Aug 28, 2018 · 0 comments

Comments

@issa-tseng
Copy link
Owner

issa-tseng commented Aug 28, 2018

This would really really centralize all the magic into App. The main issues standing in the way are:

  1. Model bind should work without an App instance. This is the worst offender. i think this is no longer a problem?
  2. The reference to this.options.app won't work the same with view subject context vs model self context.
  3. Item 1 may no longer be a problem, but now the two have diverged again due to differences in how viewmodel and subject resolve.
@issa-tseng issa-tseng changed the title Maybe: have Library provide point, unify the two and move into app Maybe: unify the two pointer()s and move into app Mar 11, 2019
@issa-tseng issa-tseng added this to the Janus v0.5 (beta release) milestone Mar 11, 2019
@issa-tseng issa-tseng removed this from the Janus v0.5 (beta release) milestone Apr 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant