Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-09-10 Thread Nikhil Mohite
Hi Team,

I have updated UI for the style guide, details as follow:
1. Resolved all UI changes suggested by  Aradhana.
2. Added labels for query-tool and tree-views icons in iconography.

PFA patch v7 for the same.

If required any changes please let me know.

Regards,
Nikhil Mohite.


On Wed, Aug 26, 2020 at 6:34 PM Aradhana Birewar <
aradhana.bire...@enterprisedb.com> wrote:

> Hi Nikhil,
>
> I was going through the style guide, I have a few observations that I
> would like to share with you.
>
>1. In the Typography section, for font size text, the Roboto font is
>not getting applied.
>
>[image: image.png]
>
>2. In the Dropdown section, the expanded dropdown image
>appears vertically stretched.
>
>[image: image.png]
>
>3. Tabs for navigating between themes need small UI modifications,
>only selected tab should have medium font-weight (font-weight: 500), others
>have normal font-weight.
>(font-weight: 400) Also selected tab bottom border should be the same
>as text width.
>
>[image: image.png]
>[image: Screenshot 2020-08-26 at 5.58.47 PM.png]
>
>4. In the Toggle Buttons section, the title should be Toggle On.
>
>[image: image.png]
>
> Everything else looks great.
>
> Thanks and Regards,
> Aradhana
>
> On Wed, Aug 26, 2020 at 4:06 PM Dave Page  wrote:
>
>> Thanks, applied with a few changes for consistency in the titles and 404
>> handling.
>>
>> On Wed, Aug 26, 2020 at 8:23 AM Nikhil Mohite <
>> nikhil.moh...@enterprisedb.com> wrote:
>>
>>> Hi Dave/Team,
>>>
>>> Updated the implementation for the set title of the page, for this, I
>>> have done a few changes in the flow.
>>> 1. Earlier we are using {% include %} for showing pages in the base
>>> page (styleguide/index.html) for a style guide but in the Django template
>>> we can't use {% block %} statement if we are adding page using "{% include
>>> %}".
>>> 2. To overcome this issue now using {% extends %} instead of  {% include
>>> %},  For this added dynamic URL generation in views so instead of loading
>>> "index.html" of style guide we are loading specific component HTML file
>>> (e.g: typography.html) and that template extends the "index.html" of the
>>> style guide.
>>>
>>> PFA v6 patch for the same.
>>>
>>> If required any changes please let me know.
>>>
>>> Regards,
>>> Nikhil Mohite.
>>>
>>>
>>> On Tue, Aug 25, 2020 at 7:09 PM Dave Page  wrote:
>>>
 Hi

 On Tue, Aug 25, 2020 at 12:49 PM Nikhil Mohite <
 nikhil.moh...@enterprisedb.com> wrote:

>
> PFA v5 patch for the same.
>
> if required any updates in this please let me know.
>

 Hmm, yeah - that doesn't work for me I'm afraid; it's mixing content
 with code.

 The titles should be defined in the HTML pages themselves. We do that
 everywhere else by including a template block like the following in the
 lowest level content:

 {% block title %}Contributing{% endblock %}

 And then in base.html:

 {% block title %}{% endblock %}

 --
 Dave Page
 Blog: http://pgsnake.blogspot.com
 Twitter: @pgsnake

 EDB: http://www.enterprisedb.com


>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EDB: http://www.enterprisedb.com
>>
>>


RM_5690_v7.patch
Description: Binary data


pgAdmin 4 commit: Remove illegal argument from trigger function in trig

2020-09-10 Thread Akshay Joshi
Remove illegal argument from trigger function in trigger DDL statement. Fixes 
#5779

Branch
--
master

Details
---
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=1bd0a0edf560057453e56f18c6a2b98708a753b3

Modified Files
--
docs/en_US/release_notes_4_26.rst   | 1 +
.../server_groups/servers/databases/schemas/tables/triggers/utils.py| 2 ++
2 files changed, 3 insertions(+)



Regarding login or reset to PGADMIN

2020-09-10 Thread vivekananda Katakam
Hi All,

I have login to web based pgadmin tool remotely and in the login screen I
was unable to login or reset my password.

When reset my password i got password reset message sent to my office email
id , but i did not receive password reset mail.

I want to change my username and password. I want to know how can I change
it.

I have installed PGadmin4 tool on RHEL Linux and running remotely.

PFA attached screenshot for more info.

Can u please suggest.

Thanks and Regards,

Vivek


Re: Regarding login or reset to PGADMIN

2020-09-10 Thread Aditya Toshniwal
Hi Vivekananda,

You need to put the SMTP configurations in config_local.py file to make it
work. Refer -
https://www.pgadmin.org/docs/pgadmin4/4.25/config_py.html#the-config-py-file
Change -

###
Mail server 
settings##
# These settings are used when running in web server mode for
confirming# and resetting passwords etc.# See:
http://pythonhosted.org/Flask-Mail/ for more infoMAIL_SERVER =
'localhost'MAIL_PORT = 25MAIL_USE_SSL = FalseMAIL_USE_TLS =
FalseMAIL_USERNAME = ''MAIL_PASSWORD = ''MAIL_DEBUG = False



On Thu, Sep 10, 2020 at 7:03 PM vivekananda Katakam <
katakamvivekana...@gmail.com> wrote:

> Hi All,
>
> I have login to web based pgadmin tool remotely and in the login screen I
> was unable to login or reset my password.
>
> When reset my password i got password reset message sent to my office
> email id , but i did not receive password reset mail.
>
> I want to change my username and password. I want to know how can I change
> it.
>
> I have installed PGadmin4 tool on RHEL Linux and running remotely.
>
> PFA attached screenshot for more info.
>
> Can u please suggest.
>
> Thanks and Regards,
>
> Vivek
>


-- 
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*

"Don't Complain about Heat, Plant a TREE"


pgAdmin 4 commit: Ensure that a valid error message should be displayed

2020-09-10 Thread Akshay Joshi
Ensure that a valid error message should be displayed for an incorrect user or 
password.

Branch
--
master

Details
---
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=ed1bd74301c7a850a40c94692d13d91bd253bdfd

Modified Files
--
web/pgadmin/__init__.py  | 5 +
web/pgadmin/authenticate/internal.py | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)