Eric Hare
Eric Hare
Just bumping this to check if there's any further assistance needed to finish this up, happy to help if so. Thanks all!
@coderabbitai Can you do an assessment of the (1) quality, (2) security robustness, and (3) merge worthiness of this PR? and please ensure you're using the LATEST commit hash: 03183dfc6a5bd15aad2925026d639729037ed7ee
@Vigtu do you mind fixing the merge conflicts? This looks great by the way!
@Vigtu same for this one, if you don't mind fixing the merge conflict i'll do a review!
@jpramos123 the failing pipeline shouldnt block the merging of this - I think we just need an approving review. There are enough frontend changes that it would be good to...
Thanks @dheechu . This is addressed in https://github.com/langflow-ai/langflow/pull/5014
@ogabrielluiz the only reason i'm thinking the astra DB changes should be combined with this PR is that it brings into sync the "secret sauce" (for lack of a better...
@ogabrielluiz i think the PR title has been updated, do things look good here now to you? Was wondering if we could get an approval to test the full CI...