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

"Observble" vs "Observer"? #489

Closed
grandsong opened this issue May 10, 2022 · 1 comment
Closed

"Observble" vs "Observer"? #489

grandsong opened this issue May 10, 2022 · 1 comment

Comments

@grandsong
Copy link

If A watch/observe B, I would name A "Observer" and B "Observble".

But, in the official document,

An observable is an object that can be initialized to watch the state of a document.

and

observable.observe(doc, (diff, before, after, local, changes) => {
  // ...
}

Why "observable"? Why not "observer"?

@ept
Copy link
Member

ept commented May 11, 2022

Yeah this class is perhaps mis-named. The proposed repository API (#486) will supersede this observable API anyway, so that will be an opportunity to clean up the naming.

@ept ept closed this as completed May 11, 2022
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

No branches or pull requests

2 participants