Problem/Motivation

The current oauth method for site registrations has issues where sites can get disconnected. Although Key will be what we use in the future, its not ready yet, so lets simplify the site connection form instead.

  1. Reformat the Acquia DAM connection screen to mimic source:
    These items should appear when navigating to admin/acquia-dam
    • Current connection status to Widen at the top of the screen if the token is filled in. - https://www.drupal.org/project/acquia_dam/issues/3546077
    • DAM Domain Name (greyed out when connected)
    • DAM Authorization Token (greyed out when successfully connected, but available to edit when connection failed)
    • Either: Disconnect Site (If domain field is filled in) or Connect Site (If domain field is missing)
    • Hide all authorization method types
  2. Do NOT remove the methods that support it, because oauth is still used for User registration)
  3. Disconnect methods should only be used for User authentication, no site disconnect
  4. Line 200 (cancelSiteRegistration first line) should be removed as sites should never ‘log out of Acquia dam’

Issue fork acquia_dam-3562132

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

rajeshreeputra created an issue. See original summary.

rajeshreeputra’s picture

Status: Active » Needs review
ankitv18’s picture

Status: Needs review » Needs work

Left few suggestion on the MR.

rajeshreeputra’s picture

Status: Needs work » Needs review
japerry’s picture

Issue summary: View changes
japerry’s picture

Status: Needs review » Needs work

japerry’s picture

Looks good! Merged.

japerry’s picture

Status: Needs work » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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