On Thu, Jul 2, 2020 at 5:41 PM Khushboo Vashi <
khushboo.va...@enterprisedb.com> wrote:

>
>
> On Thu, Jul 2, 2020 at 5:33 PM Murtuza Zabuawala <
> murtuza.zabuaw...@enterprisedb.com> wrote:
>
>> Hi Khushboo,
>>
>> I am already on the latest code :)
>>
> Then your patch will not work. :)
>

It is applying and I'm able to see the diff,

------

(venv3_pg) murtuza@ubuntu:~/projects/pgadmin4 (master)$ git apply
browser_regression.diff
(venv3_pg) murtuza@ubuntu:~/projects/pgadmin4 (master)$ gs
On branch master
Your branch is up to date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

modified:   web/pgadmin/browser/static/js/browser.js


-----

(venv3_pg) murtuza@ubuntu:~/projects/pgadmin4 (master)$ git diff
diff --git a/web/pgadmin/browser/static/js/browser.js
b/web/pgadmin/browser/static/js/browser.js
index 9db31de59..dc6c26920 100644
--- a/web/pgadmin/browser/static/js/browser.js
+++ b/web/pgadmin/browser/static/js/browser.js
@@ -278,16 +278,16 @@ define('pgadmin.browser', [
       /* Add hooked-in panels by extensions */
       var panels = JSON.parse(pgBrowser.panels_items);
       _.each(panels, function(panel) {
-        if (panel.isIframe) {
+        if (panel.is_iframe) {
           pgBrowser.frames[panel.name] = new pgBrowser.Frame({
             name: panel.name,
             title: panel.title,
             icon: panel.icon,
             width: panel.width,
             height: panel.height,
-            showTitle: panel.showTitle,
-            isCloseable: panel.isCloseable,
-            isPrivate: panel.isPrivate,
+            showTitle: panel.show_title,
+            isCloseable: panel.is_closeable,
+            isPrivate: panel.is_private,
             url: panel.content,
           });
         } else {
@@ -297,12 +297,12 @@ define('pgadmin.browser', [
             icon: panel.icon,
             width: panel.width,
             height: panel.height,
-            showTitle: panel.showTitle,
-            isCloseable: panel.isCloseable,
-            isPrivate: panel.isPrivate,
+            showTitle: panel.show_title,
+            isCloseable: panel.is_closeable,
+            isPrivate: panel.is_private,
             content: (panel.content) ? panel.content : '',
             events: (panel.events) ? panel.events : '',
-            canHide: (panel.canHide) ? panel.canHide : '',
+            canHide: (panel.can_hide) ? panel.can_hide : '',
           });
         }
       });




>
>> *commit* ec30e6ace78a813eb5215efc5c1f0e94582ffc12 (HEAD -> master,
>> origin/master, origin/HEAD)
>>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> *EDB*
>> *POWER TO POSTGRES*
>> https://www.edbpostgres.com
>>
>>
>> On Thu, Jul 2, 2020 at 5:25 PM Khushboo Vashi <
>> khushboo.va...@enterprisedb.com> wrote:
>>
>>>
>>>
>>> On Thu, Jul 2, 2020 at 5:16 PM Murtuza Zabuawala <
>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>
>>>> Hi,
>>>>
>>>> There was a regression where we have changed the panel variables in
>>>> menu.py but we did not change their respective reference in JS which
>>>> is causing issues while loading wcDocker Panel/Frames which are defined in
>>>> python side.
>>>>
>>>> This has been fixed, please get the latest pgAdmin code.
>>>
>>>> Ref: 830ac1cc9177c8b9ebed2304924cec1db22e45c3
>>>> <https://git.postgresql.org/gitweb/?p=pgadmin4.git;a=commit;h=830ac1cc9177c8b9ebed2304924cec1db22e45c3>
>>>>
>>>>
>>>> --
>>>> Regards,
>>>> Murtuza Zabuawala
>>>> *EDB*
>>>> *POWER TO POSTGRES*
>>>> https://www.edbpostgres.com
>>>>
>>>

Reply via email to