Re: [pgAdmin4][RM2136] Include "?" on all routes that are loaded

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: [pgAdmin4][RM2136] Include "?" on all routes that are loaded
Дата
Msg-id CAFOhELexXLHtDQjKu+MowoYFUYQQyiKX8W8A3DgrgZdsq=Dqpw@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][RM2136] Include "?" on all routes that are loaded  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Ответы Re: [pgAdmin4][RM2136] Include "?" on all routes that are loaded  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения
Hi Aditya,

- Some of the svg files (connect.svg, disconnect.svg in Query Tool) are left.
- The variable name (INTERNAL_VERSION_PARAM) in the config can be renamed as APP_VERSION_PARAM
- The value of the INTERNAL_VERSION_PARAM can be 'ver' or 'v' instead of 'intver'.

Except above the patch looks good to me.

Thanks,
Khushboo

On Thu, Jul 26, 2018 at 5:54 PM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch for RM2136 which will add "?<ver_param>=<ver>" to all the static files requested and will help in cache busting on upgrading pgAdmin4.
<ver_param> name is configurable and is present in config.py-INTERNAL_VERSION_PARAM.
The value <ver> is taken as APP_VERSION_INT (which is updated with every release).
Setting INTERNAL_VERSION_PARAM value to blank or None will disable the version param add.
New parameter INTERNAL_VERSION_EXTN (tuple) is also added to specify the file extensions for which the version param to be added. I have added a few for now.
Test cases included.

Kindly review.

--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Xuri Gong
Дата:
Сообщение: [pgAdmin4][patch]: Feature #1407 - Support map view for PostGIS query result sets
Следующее
От: Akshay Joshi
Дата:
Сообщение: [pgAdmin4][Patch]: RM #3511 Passwords not stored for ssh tunnel connections