Once you have an Instagram client set up with the module there is no way to remove this client's key in the UI. We need to provide a method to delete the Instagram credential variables.

CommentFileSizeAuthor
#3 isf-2413281-remove_feed_api_key-3.patch3.43 KBdysproseum
#1 isf_remove_feed_api_key-2413281-1.patch3.32 KBmarkie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markie’s picture

markie CreditAttribution: markie commented
Status: Active » Needs review
FileSize
isf_remove_feed_api_key-2413281-1.patch3.32 KB

Updated logic so variable is deleted and form is editable.

kepford’s picture

kepford CreditAttribution: kepford commented

This patch applies cleanly to develop branch but the installation process is currently not working. Specifically there isn't a proper menu for the module. I believe this is related to this issue.

dysproseum’s picture

dysproseum CreditAttribution: dysproseum commented
FileSize
isf-2413281-remove_feed_api_key-3.patch3.43 KB

Submitted patch in #1 did not indicate that authentication was successful. The API key was stored in the correct variable, but settings form was not updated.

Attaching updated patch that fixes that logic.

kepford’s picture

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

API key is removed and patch applies.

dysproseum’s picture

dysproseum CreditAttribution: dysproseum commented
Status: Reviewed & tested by the community » Fixed

Patch committed.

Status: Fixed » Closed (fixed)

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