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

Prepare to implement tracked storage, RFC#669 #20761

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

NullVoxPopuli
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli commented Sep 30, 2024

Motivation here is that I want to stop folks from using https://github.com/ember-polyfills/ember-tracked-storage-polyfill (and swap its implementation to use the real implementation (here, hopefully)


This doesn't make sense to continue work on until

lands -- because I don't really want to continue the fake-glimmer packages we have in @ember/-internals/metal

Additionally, this RFC, https://rfcs.emberjs.com/id/0669-tracked-storage-primitive#re-implementing-tracked-with-storage, kinda just likes like Starbeam concepts, so it should be super easy:

export function trackedStorage(...) {
  return Cell(...);
}

export function getValue(cell) {
  return cell.current;
}

export function setValue(cell, value) {
  return cell.set(value);
}

ez

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.

1 participant