Ok the commit is reverted :(

On Thu, Jul 2, 2020 at 5:45 PM Murtuza Zabuawala <
murtuza.zabuaw...@enterprisedb.com> wrote:

>
>
>
> 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