Kijong Han

Results 12 comments of Kijong Han

Hey @HofmeisterAn, thoughts on trying to simplify this method override? https://github.com/testcontainers/testcontainers-dotnet/blob/f746adfe1edb7bd8e3296fcca4de19b2e0352ecc/src/Testcontainers.Module.Example/ExampleTestcontainersBuilder.cs#L35-L47 It looks like many module implementations of the `TestcontainersBuilder` abstract class will have to have similar switch statements. Having...

hi @akshay-joshi - could you please look into reapproving the workflows? rebased from upstream

> With this patch, the browser window gets blank if I update the server group name. Please take a look at the console logs. > > thanks for testing the...

I've encountered a few other issues testing these changes - updating any child nodes (anything under server groups) results in the nodes being placed in the root directory node (the...

> > > With this patch, the browser window gets blank if I update the server group name. Please take a look at the console logs. > > > >...

> Hi @KijongHan, are you planning to work on the reported issue, or should I close this PR? hey @khushboovashi sorry just got back from my holiday. Ill try working...

hi @khushboovashi - many thanks for the patience on this one. I've changed my approach for the fix on this one. Instead of trying to make the reorder work with...

hi @akshay-joshi, do you think I could get some help with getting some eyes on these changes? thank you!

Hi there, I made a PR for a fix against this issue here https://github.com/pgadmin-org/pgadmin4/pull/8547

looks like this might be a possibility now as a result of this merged PR to the forked data grid library? https://github.com/pgadmin-org/react-data-grid/pull/7