Re: [pgadmin-hackers][patch] Move to Alembic migration system - Mailing list pgadmin-hackers

From Ashesh Vashi
Subject Re: [pgadmin-hackers][patch] Move to Alembic migration system
Date
Msg-id CAG7mmoxoP+iQuy_03q3A4SHskdwMjyU3AwR52SwYpZ_98mCRsg@mail.gmail.com
Whole thread Raw
In response to Re: [pgadmin-hackers][patch] Move to Alembic migration system  (Joao Pedro De Almeida Pereira <jdealmeidapereira@pivotal.io>)
Responses Re: [pgadmin-hackers][patch] Move to Alembic migration system  (Joao Pedro De Almeida Pereira <jdealmeidapereira@pivotal.io>)
List pgadmin-hackers
Hi Joao & Oliver,

On Fri, Apr 21, 2017 at 3:39 AM, Joao Pedro De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Hackers,

@Ashesh thanks for the feedback

Here is the reviewed patch with the suggestions of Ashesh.

Disclaimer: We added a new patch file with the changes 
 
I have made some more changes to the patch.
- 'with app.app_context(..)' statement was not required in the 'web/pgadmin/__init__.py' as we're already doing that in the do_upgrade function.
- We also need to create other directories (i.e. sessions, storage, directory containing the log-file) during the setup/running the application (if not exists).
- Added proper check in the pgAdmin4.wsgi file (if configuration file exists, or not)

Please review it.

-- Thanks, Ashesh

Attachment

pgadmin-hackers by date:

Previous
From: Khushboo Vashi
Date:
Subject: Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM #2339 - Horizontalscrolling in the treeview
Next
From: Murtuza Zabuawala
Date:
Subject: Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue withdisplaying bigint[] values in query tool