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.

Sign up for GitHub

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

Jump to bottom

[SPEC] APIs for managing per-stream states in the UI #12991

Closed
andyjih opened this issue May 18, 2022 · 1 comment · Fixed by #13835
Closed

[SPEC] APIs for managing per-stream states in the UI #12991

andyjih opened this issue May 18, 2022 · 1 comment · Fixed by #13835
Assignees
@cgardens
Labels
team/platform-move

Comments

@andyjih
Copy link
Contributor

andyjih commented May 18, 2022 •
edited
Loading

No description provided.

@andyjih andyjih changed the title [SPEC] APIs for showing per-stream states in the UI [SPEC] APIs for managing per-stream states in the UI May 18, 2022
@benmoriceau benmoriceau self-assigned this May 23, 2022
@timroes
Copy link
Collaborator

timroes commented May 26, 2022

@benmoriceau @andyjih I don't think we'd want a "per-stream" API from the UI perspective. Given the designs we currently have, this would still be all one form, and I'd expect us to continue calling the web_backend/connections/update API, which gets the information about all the streams and saves them (and also resets them as I described in #12741).

@cgardens cgardens mentioned this issue Jun 16, 2022
Merged
@benmoriceau benmoriceau assigned cgardens and unassigned benmoriceau Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cgardens cgardens

Labels
team/platform-move
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update api for per stream airbytehq/airbyte
4 participants
@andyjih @timroes @benmoriceau @cgardens