Chandrasekharan M

Results 30 comments of Chandrasekharan M

>I would recommend to use the older API for now. @shcheklein thanks for the information, I did go through the docs a bit but can you elaborate on what you...

#1089 was already merged and hence closing this redundant PR. Please reopen if required - @jagadeeswaran-zipstack @tahierhussain

> ## [![Quality Gate Failed](https://camo.githubusercontent.com/5333b8e2ff49bd5efdc6d2efd30ec08d7ac8d67c5e80190d03b6c2cb29a5de8b/68747470733a2f2f736f6e6172736f757263652e6769746875622e696f2f736f6e6172636c6f75642d6769746875622d7374617469632d7265736f75726365732f76322f636865636b732f5175616c6974794761746542616467652f71672d6661696c65642d323070782e706e67)](https://sonarcloud.io/dashboard?id=Zipstack_unstract&pullRequest=567) **Quality Gate failed** > Failed conditions ![](https://camo.githubusercontent.com/ee78c91cde137cf4668f72b8e1247dadaaeb117c99c74bed0f21b1681b517491/68747470733a2f2f736f6e6172736f757263652e6769746875622e696f2f736f6e6172636c6f75642d6769746875622d7374617469632d7265736f75726365732f76322f636f6d6d6f6e2f6661696c65642d313670782e706e67) [15.6% Duplication on New Code](https://sonarcloud.io/component_measures?id=Zipstack_unstract&pullRequest=567&metric=new_duplicated_lines_density&view=list) (required ≤ 3%) > > [See analysis details on SonarCloud](https://sonarcloud.io/dashboard?id=Zipstack_unstract&pullRequest=567) I couldn't clearly...

Closing this PR since we agreed not to introduce ES for this usecase. Will refer this PR in subsequent metrics related changes instead

> @chandrasekharan-zipstack Migration looks good. > > One clarification though. In the screenshots provided, I see that `fileExtensions` array is empty after migration. Please check if this is a miss...

> LGTM- Is this tablesettings oss fetaure? @muhammad-ali-e I believe its a cloud / enterprise feature

Hello @haluwong Since `v0.93.0` there has been some changes in how we store data. Can you attempt to follow [this migration guide](https://github.com/Zipstack/unstract/blob/125a9304cc30c88893a552071b1bb0611b035073/backend/migrating/v2/README.md#L18) to move existing data into the new structure...

Closing this PR to retain these pills and remain transparent. Can be reopened later if necessary

Code duplication is due to the v2 migration changes also being present in this PR

> LGTM. Just to confirm, we're only saving the result in the database when the API is called with specific parameters, right? @muhammad-ali-e in case of workflow execution, we'll never...