An Phi
An Phi
@YannanGao-gs as discussed, we will deprioritize this, I'm putting it in the backlog
@MauricioUyaguari @YannanGao-gs I'll move this to `On Deck` for now, we need a lot of discussion around this kind of feature.
Can we finish this one :)
This will probably require changes in https://github.com/finos/legend-engine to handle newline characters. Similar work has been done for relational mapping test in https://github.com/finos/legend-engine/pull/238 https://github.com/finos/legend-engine/pull/245 https://github.com/finos/legend-engine/pull/252
For the limiting the result being shown by 100, I wonder if we need some visual indicator or ways to show `View More`, what do you think?
I also tried to use `@jest/global` - https://jestjs.io/blog/2020/05/05/jest-26#a-new-way-to-consume-jest---jestglobals - I have tried to make it work with the following patch [0001-use-jest-globals.patch.txt](https://github.com/finos/legend-studio/files/8575402/0001-use-jest-globals.patch.txt) but there are a few problems: - There are...
I have updated the codebase to use `@jest/globals` but still stuck with `injectGlobals`. Seems like that flag has more implications than just removing the globals, I start seeing errors in...
Workaround regarding issue with `uuid` has been resolved by https://github.com/finos/legend-studio/pull/1453
`Jest` has this weird problem around `Promise`, sometimes, `.mockResolvedValue()` does not work as intended See https://github.com/facebook/jest/issues/2549 See https://github.com/facebook/jest/issues/6645 See https://github.com/facebook/jest/issues/2549 Maybe it's not a bad idea to switch to another...
@YannanGao-gs If you already linked this PR to an issue which is already milestoned (`8.0.0` in this case), let's not milestone the PR, this creates duplicates in our project board....