Hello Murtuza, After some review of the code we do have some questions: - Do we really need to have the shortcuts in the backend?(I already asked this in Khushboo's patch) - Do we need to have keyAction function(in direct.js) inside this Javascript file or can we extract that an create some tests around it? - Were there any tests created for these new shortcuts?
Thanks Joao On Thu, Feb 8, 2018 at 6:33 AM Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi, > > PFA patch to make shortcut configurable in debugger module. > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > >
