Skip to content

POC: Implement basic endpoint for fetching workflow executions#6

Merged
stevekinney merged 1 commit into
mainfrom
workflow-store
Jul 9, 2021
Merged

POC: Implement basic endpoint for fetching workflow executions#6
stevekinney merged 1 commit into
mainfrom
workflow-store

Conversation

@stevekinney

Copy link
Copy Markdown
Contributor

POC: Implement basic endpoint for fetching workflow executions from containerized Temporal

This is a very naive implementation of an endpoint that can talk to the containerized version of Temporal to fetch a list of workflow executions.

The types need work and there are clearly more query parameters that we'll need to support. There are edge cases left unhandled. But, it's a quick proof of concept that will allow us to work against the fixtures in the samples-go repository as we build out some of the pages.

To-do:

  • Make this work with single-binary, web-v2 server.
  • Figure out how we're going to bundle this with the single binary (we may choose not to and only use these endpoints in the cloud version).
  • Add support for the rest of the query parameters currently supported.
  • Add better handling.
  • Refine types.
  • Add support for respecting next page tokens.
  • (Probably much more)

…ontainerized Temporal

This is a very naive implementation of an endpoint that can talk to the containerized version of Temporal to fetch a list of workflow executions.

The types need work and there are clearly more query parameters that we'll need to support. There are edge cases left unhandled. But, it's a quick proof of concept that will allow us to work against the fixtures in the `samples-go` repository as we build out some of the pages.

To-do:

- Make this work with single-binary, web-v2 server.
- Figure out how we're going to bundle this with the single binary (we may choose not to and only use these endpoints in the cloud version).
- Add support for the rest of the query parameters currently supported.
- Add better handling.
- Refine types.
- Add support for respecting next page tokens.
- (Probably much more)
@feedmeapples

Copy link
Copy Markdown
Contributor

approving with a plan to eventually build the typescript types from API

Comment thread tsconfig.json
"$lib/*": ["src/lib/*"],
"$components/*": ["./src/components/*"],
"$utilities/*": ["./src/utilities/*"]
"$components/*": ["src/components/*"],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

afaik eventually we want to have src/<feature-name>/components structuring?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SvelteKit mostly does the feature-based folder structure with routes, src/components is for common components that are used throughout the application.

@stevekinney
stevekinney merged commit 8ea9074 into main Jul 9, 2021
@stevekinney
stevekinney deleted the workflow-store branch July 9, 2021 18:12
tegan-temporal added a commit that referenced this pull request Jul 8, 2026
- activity-options-update-drawer: default backoffCoefficient to 2 (server
  default) instead of 0 — Temporal rejects < 1, and the full field mask
  always submits it. (finding 2)
- getFirstWholeNumberUnit: add overloads so passing a defaultUnit returns
  a non-undefined label; narrow initialTimeoutUnit to DefaultUnits and drop
  4 now-dead `?? 'second(s)'` fallbacks. (finding 4)
- dangerfile: match changed files by normalized path equality instead of
  fragile bidirectional substring (missed regressions / false-blocked
  suffix collisions). (finding 5)
- timeline-graph-row: guard divide-by-zero so an undefined workflowDistance
  yields 0, not Infinity, as an SVG coordinate. (finding 7)
- rename MostRecentWOrkflowVersionStamp -> MostRecentWorkflowVersionStamp
  typo (now load-bearing imported surface). (finding 9)

Deferred: #3 (requestFromAPI -> Promise<T|undefined> is a 128-call-site
refactor, its own PR), #6 (documented zod input/output boundary), #8
(pre-existing on main, not this change).
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.

2 participants