Problem/Motivation

After saving settings form, there is no way to confirm that API settings are correct

Proposed resolution

Create a button in settings page, which on click goes to a custom route which tests the `apiHealthStatus` with the api settings saved in the settings form

CommentFileSizeAuthor
#6 screenshot-drupal10.ddev_.site-2023.05.16-14_32_20.png39.3 KBkpoornima
#5 Screenshot from 2023-05-15 18-59-492.png70.65 KBshiv_sharma
#4 Screenshot from 2023-05-15 19-02-062.png69.41 KBshiv_sharma

Issue fork nginx_proxy_manager_connector-3359357

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

shashikanth171 created an issue. See original summary.

shiv_sharma’s picture

shiv_sharma commented
Assigned: Unassigned » shiv_sharma

shashikanth171 opened merge request !19

shiv_sharma’s picture

shiv_sharma commented
StatusFileSize
new Screenshot from 2023-05-15 19-02-062.png69.41 KB

Added screenshot for reference

shiv_sharma’s picture

shiv_sharma commented
Assigned: shiv_sharma » Unassigned
Status: Active » Needs review
Issue tags: +api validation
StatusFileSize
new Screenshot from 2023-05-15 18-59-492.png70.65 KB
kpoornima’s picture

kpoornima commented
StatusFileSize
new screenshot-drupal10.ddev_.site-2023.05.16-14_32_20.png39.3 KB

@shiv_sharma After testing MR 19 . I have few doubts
1)Only API Base URL field is need for connect to Nginx proxy Manger
2) With out Entering data into Email and password filed and clicked on check box of " Validate Connection" getting this message "API Base URL is Connected"
Please refer screenshot
connection

shiv_sharma’s picture

shiv_sharma commented

yes @kpoornima only API Base URL fiels is required to test connection.
return $this->httpClient->request('GET', $api_url)->getBody()->getContents();

kpoornima’s picture

kpoornima commented
Status: Needs review » Reviewed & tested by the community

Thanks @Shiv_sharma

shiv_sharma’s picture

shiv_sharma commented
Status: Reviewed & tested by the community » Needs review
shiv_sharma’s picture

shiv_sharma commented
Status: Needs review » Reviewed & tested by the community
nishant’s picture

nishant
Primary language English
commented

I have tested the MR19 with drupal 9.5.8,
Php version 8.0.28 and it's working fine for me.

  • shashikanth171 committed e1f96abe on 1.x 
    Issue #3359357 by Shiv_Sharma, shashikanth171, kpoornima, Nishant: Test...
shashikanth171’s picture

shashikanth171 commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -api validation

Merged MR. Thank you, @kpoornima @shiv_sharma @nishant.

@nishant and @shiv_sharma. Please add screenshots for any review wherever it's relevant.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.