Add checkpoint UI by Karakatiza666 · Pull Request #6065 · feldera/feldera · GitHub
Skip to content

Add checkpoint UI#6065

Draft
Karakatiza666 wants to merge 1 commit intomainfrom
checkpoint-ui
Draft

Add checkpoint UI#6065
Karakatiza666 wants to merge 1 commit intomainfrom
checkpoint-ui

Conversation

@Karakatiza666
Copy link
Copy Markdown
Contributor

Related PR: #6003

Describe Manual Test Plan

Checklist

  • Unit tests added/updated
  • Integration tests added/updated
  • Documentation updated
  • Changelog updated

Breaking Changes?

Mark if you think the answer is yes for any of these components:

Describe Incompatible Changes

Signed-off-by: Karakatiza666 <bulakh.96@gmail.com>
Copy link
Copy Markdown

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

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

This web-console behavior change still needs frontend coverage; see inline.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This PR adds a substantial new web-console behavior surface: checkpoint polling, new checkpoint status/indicator UI, drawer state changes, and new manager calls. Per our standing web-console rule, that needs frontend tests before approval. A good minimal bar here would be Vitest/@testing-library coverage that exercises the checkpoint activity rendering and the new checkpoints drawer / request-checkpoint flow, with the API calls mocked via usePipelineManager.

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