Problem/Motivation

There's no need to ask for both full URLs for API and OAuth, as there are only two possible versions, depending on whether you're in sandbox mode or not.

Proposed resolution

Replace the two URLs fields in the settings form by a single "Sandbox" checkbox.

Remaining tasks

Validate MR.

User interface changes

The 2 URLs fields in settings form will be replaced by a single "Sandbox" checkbox:

CaptchEtat sandbox mode

Data model changes

The 2 URLs text fields in schema will be replaced by a single boolean one for the "Sandbox" state.

Issue fork captchetat-3450075

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:

Comments

mably created an issue. See original summary.

mably’s picture

Need to have this issue (CaptchetatService) merged before pushing this issue's MR.

ewehbe’s picture

Status: Active » Reviewed & tested by the community

Need to be reviewed

ewehbe’s picture

Status: Reviewed & tested by the community » Needs review

mably’s picture

Issue summary: View changes
StatusFileSize
new90.85 KB
mably’s picture

StatusFileSize
new78.24 KB
mably’s picture

Issue summary: View changes
mably’s picture

@ewehbe it should be ok to merge: it has been tested successfully with sandbox and production credentials.

ewehbe’s picture

Status: Needs review » Reviewed & tested by the community

  • ewehbe committed 35da6142 on 2.0.x authored by mably
    Issue #3450075 by mably: Replace API and OAuth URLs in settings by a...
ewehbe’s picture

Status: Reviewed & tested by the community » Fixed
ewehbe’s picture

Status: Fixed » Closed (fixed)
mably’s picture

Assigned: mably » Unassigned