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

feat: DH-16737 Add ObjectManager, useWidget hook (#2030) #2056

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

mofojed
Copy link
Member

@mofojed mofojed commented Jun 4, 2024

  • Hook for loading a widget that makes it easy to use
  • ObjectManager context allows for implementation to handle loading
    the object
  • On Core side, we have a very simple ObjectManager, as we just have
    one connection
  • On Enterprise side, the ObjectManager provided to the context
    manager will need to handle fetching from queries, and will be able to
    handle more scenarios (such as when a query is restarting)
  • Use with feat: Use useWidget hook to load widgets deephaven-plugins#502

- Hook for loading a widget that makes it easy to use
- `ObjectManager` context allows for implementation to handle loading
the object
- On Core side, we have a very simple `ObjectManager`, as we just have
one connection
- On Enterprise side, the `ObjectManager` provided to the context
manager will need to handle fetching from queries, and will be able to
handle more scenarios (such as when a query is restarting)
- Use with deephaven/deephaven-plugins#502
@mofojed mofojed requested a review from mattrunyon June 4, 2024 19:32
@mofojed mofojed self-assigned this Jun 4, 2024
@mofojed mofojed enabled auto-merge (squash) June 4, 2024 19:34
@mofojed mofojed merged commit dbf613b into deephaven:main Jun 4, 2024
10 checks passed
@mofojed mofojed deleted the use-object-fetch-main branch June 4, 2024 19:46
@github-actions github-actions bot locked and limited conversation to collaborators Jun 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants