Attached is the updated patch to resolve the raised concerns.
On Fri, Jul 24, 2020 at 5:37 PM Dave Page <dpage@pgadmin.org> wrote:
Hi
Some further thoughts/comments:
- There should be a comment block in every source file, containing the copyright info etc.
Done
- Looks like the "new-" in "new-dashboard-graphs" is probably a development artefact and should be removed?
Yeah, missed that. Done.
- We should probably use .jsx as the extension for React JS files, to help editors (and our brains). I'm guessing in the future we may have some .tsx as well.
Umm. OK. Done.
- Not sure why there's a package-lock.json file in there - we use Yarn :-)
I'm not sure where the file came from. I checked my patch twice but no sign of package-lock. :P
On Thu, Jul 23, 2020 at 3:59 PM Dave Page <dpage@pgadmin.org> wrote:
Very nice. One quick issue that I immediately saw - if the dashboard is not the top-most tab when pgAdmin opens, then when you've selected something on the treeview and then click on the dashboard tab, the graphs don't populate until you select a different object.
I'll play some more tomorrow when I have more time.
