Skip to content

feat: add streaming checklists #126

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
Merged
merged 14 commits into from
Dec 18, 2024
Merged

feat: add streaming checklists #126

merged 14 commits into from
Dec 18, 2024

Conversation

mariobuikhuizen
Copy link
Contributor

No description provided.

@mariobuikhuizen mariobuikhuizen force-pushed the feat_streaming_checklists branch 2 times, most recently from 2282ed2 to c4681d3 Compare November 18, 2024 09:03
@mariobuikhuizen mariobuikhuizen force-pushed the feat_streaming_checklists branch from c4681d3 to c35b5b4 Compare November 18, 2024 09:10
@mariobuikhuizen mariobuikhuizen marked this pull request as ready for review November 22, 2024 14:28
ConnorDang0
ConnorDang0 reviewed Dec 8, 2024
View reviewed changes
Copy link
Contributor

@ConnorDang0 ConnorDang0 left a comment

Choose a reason for hiding this comment

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

Can we also add an implementation for hitting the reloadStreamingChecklist endpoint?

@mariobuikhuizen
Copy link
Contributor Author

mariobuikhuizen commented Dec 12, 2024 •
edited
Loading

This is blocked until there is a way to link a Checklist to an Asset

alkasm
alkasm reviewed Dec 12, 2024
View reviewed changes
mariobuikhuizen and others added 2 commits December 13, 2024 17:39
@mariobuikhuizen @alkasm
fix wrong seconds used
1528a3e 
Co-authored-by: Alexander Reynolds <alex@theory.shop>
@mariobuikhuizen
rename to integration_rids
bb39bc7
@mariobuikhuizen mariobuikhuizen force-pushed the feat_streaming_checklists branch from b5ab3eb to bb39bc7 Compare December 13, 2024 19:02
@mariobuikhuizen mariobuikhuizen force-pushed the feat_streaming_checklists branch from dd466af to c23c2f6 Compare December 17, 2024 19:51
@jackparmer jackparmer merged commit 3093a4c into main Dec 18, 2024
5 checks passed
@jackparmer jackparmer deleted the feat_streaming_checklists branch December 18, 2024 23:18
@nominal-bot nominal-bot mentioned this pull request Dec 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alkasm alkasm alkasm left review comments

@ConnorDang0 ConnorDang0 ConnorDang0 left review comments

@jackparmer jackparmer jackparmer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mariobuikhuizen @jackparmer @alkasm @ConnorDang0