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

Fetch DHE js api from server #79

Open
Tracked by #17
bmingles opened this issue Aug 1, 2024 · 0 comments · Fixed by #91, #92 or #103
Open
Tracked by #17

Fetch DHE js api from server #79

bmingles opened this issue Aug 1, 2024 · 0 comments · Fixed by #91, #92 or #103
Assignees

Comments

@bmingles
Copy link
Collaborator

bmingles commented Aug 1, 2024

We need to fetch the DHE JS api from a running server.

  • DHE: split out npm packages containing necessary types
    • DH-17425: web-client-ui minimal monorepo
    • DH-16725: @deephaven-enterprise/jsapi-types is related but not strictly necessary for this task).
  • Seems likely all runtime code should come from the server while types will need to be installed from a package to facilitate dev of the vscode extension. aka. types aren't needed for runtime.
  • new npm package to encapsulate the fetching of the api from a server (mentioned in jsapi - improvements for 3rd party consumers deephaven-core#5537)
@bmingles bmingles self-assigned this Aug 1, 2024
bmingles added a commit that referenced this issue Aug 6, 2024
bmingles added a commit that referenced this issue Aug 7, 2024
bmingles added a commit that referenced this issue Aug 7, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
Refactored a few things
- Fixed base DhService + constructor type in preparation for DHE support
- Installed @deephaven/jsapi-types and deleted shim types
- Tweaked tmp storage to store to src/dist root and to store jsapi by
server url
- Enabled `no-unused-vars` linting rule

supports #79
@bmingles bmingles linked a pull request Aug 8, 2024 that will close this issue
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 8, 2024
bmingles added a commit that referenced this issue Aug 9, 2024
bmingles added a commit that referenced this issue Aug 9, 2024
bmingles added a commit that referenced this issue Aug 9, 2024
bmingles added a commit that referenced this issue Sep 13, 2024
bmingles added a commit that referenced this issue Sep 13, 2024
bmingles added a commit that referenced this issue Sep 13, 2024
bmingles added a commit that referenced this issue Sep 13, 2024
bmingles added a commit that referenced this issue Sep 13, 2024
bmingles added a commit that referenced this issue Sep 16, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
bmingles added a commit that referenced this issue Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment