Regarding this patch we refactored the Javascript code so that is lives in a different file and added some tests.
Also we found an issue with karma-jasmine that does not allow us to use jasmine 3.1 yet. You can find attached a patch that reverts that commit.
Sounds good, but neither patch will apply (in fact, the Jasmine one looks entirely backwards). One of the error messages was changed in Murtuza's patch, and wasn't reflected in your update for example.
Can you rebase please?
Thanks.
Thanks
Victoria && Joao
On Mon, Mar 12, 2018 at 4:46 PM Dave Page <dpage@pgadmin.org> wrote:
PFA patch to add service parameter in server dialog.
- Docs updated
- Test case added for Service ID parameter
Please note,
I have extracted Connection class and Server manager class from our own custom Psycopg2 driver module.
Patch also covers RM#3120
This patch seems a little confused. The "Service" and "Service ID" fields from pgAdmin 3 are very different things. The Redmine ticket seems to be asking for the Service field (the pg_service.conf service name), *not* Service ID (the operating system's service ID, used to start/stop the database server service).