Problem/Motivation

To facilitate better user experience, we can associate an oauth consumer with the preview site so it will be easier to get the proper credentials for that site at another time.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

A new field will be added to the preview site configuration entity that is a reference to an oAuth Consumer entity.
This field will be optional.
This field will be added during new installs of the module.
This field will also be added for existing installations of the decoupled-preview module that upgrade to the most recent version.

API changes

Data model changes

Issue fork decoupled_preview-3391729

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

coby.sher created an issue. See original summary.

mitchellmarkoff made their first commit to this issue’s fork.

brianperry made their first commit to this issue’s fork.

mitchellmarkoff opened merge request !12

brianperry’s picture

brianperry
he/him
commented
Status: Active » Needs review

@mitchellmarkoff made one small change here to store the client_id for the associated consumer rather than the label. Generally an id is better for stuff like this, but the label would also be problematic here as we can't assume that it will be unique.

mitchellmarkoff’s picture

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

brianperry
he/him
commented
Status: Reviewed & tested by the community » Fixed

brianperry opened merge request !13

brianperry opened merge request !14

Status: Fixed » Closed (fixed)

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