Problem/Motivation

The settings screen contains a checkbox labeled "Show a link to the privacy policy". This doesn't seem to do anything. When looking through the code, the HTML snippet it adds contains a link to the URL https://akismet.com/web-service-privacy-policy, which doesn't actually exist. It would seem this is actually a quick and dirty find-replace from the similar functionality in the Mollom module. AFAICT, Akismet does not actually have such a requirement.

Proposed resolution

Remove the setting and its associated functionality. Also remove any other reference to the privacy link; e.g. the readme references it as well.

Remaining tasks

  • Create merge request
  • Consolidate README (see comment #6).
  • Review
  • Commit

User interface changes

The configuration option "Show a link to the privacy policy" disappears.

API changes

None.

Data model changes

None.

Issue fork akismet-3075142

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

eelkeblok created an issue. See original summary.

eelkeblok’s picture

eelkeblok
he/him
Primary language Dutch
Location Netherlands 🇳🇱
commented
Issue summary: View changes
eelkeblok’s picture

eelkeblok
he/him
Primary language Dutch
Location Netherlands 🇳🇱
commented
Title: Checkbox "Show a link to the privacy policy" does not do anything » Remove configuration "Show a link to the privacy policy"

eelkeblok opened merge request !9

eelkeblok’s picture

eelkeblok
he/him
Primary language Dutch
Location Netherlands 🇳🇱
commented
Status: Active » Needs review
eelkeblok’s picture

eelkeblok
he/him
Primary language Dutch
Location Netherlands 🇳🇱
commented

Trying to fixed conflicts made me realise we lost some content from the README through the latest few issues updating it. I committed the version from this branch in addition of the new README.md, but they need to be consolidated before this can ge merged.

eelkeblok’s picture

eelkeblok
he/him
Primary language Dutch
Location Netherlands 🇳🇱
commented
Issue summary: View changes
eelkeblok’s picture

eelkeblok
he/him
Primary language Dutch
Location Netherlands 🇳🇱
commented
Version: 8.x-1.x-dev » 2.x-dev
Status: Needs review » Needs work

  • eelkeblok committed f7a8d112 on 2.x 
    Merge branch '3075142-checkbox-show-a' into '2.x'

Issue #3075142:...

  • eelkeblok committed c621af74 on 2.x 
    Issue #3075142 by eelkeblok: Remove configuration "Show a link to the...