Problem/Motivation

Would like to allow hosting of new app-ads.txt in addition to ads.txt.

Specification: https://iabtechlab.com/wp-content/uploads/2019/03/app-ads.txt-v1.0-final...

Steps to reproduce

No steps to reproduce, module doesn't currently provide this functionality.

Proposed resolution

Add additional configuration field and controller endpoint for app-ads.txt.

Remaining tasks

Review and merge provided patch.

User interface changes

New app-ads.txt field on edit form.

API changes

New route /app-ads.txt. Added logic to throw NotFoundHttpException if configuration is empty.

Data model changes

New field in configuration.

CommentFileSizeAuthor
adstxt_app_ads.patch5.11 KBnbabb

Comments

nbabb created an issue. See original summary.

marcelovani’s picture

marcelovani
Primary language Portuguese, Brazil
Location London
commented
Status: Active » Needs review

  • marcelovani committed f97606b on 3176836-app-txt authored by nbabb 
    Issue #3176836 by nbabb, marcelovani: Add support for app-ads.txt

  • marcelovani committed 3dec611 on 3176836-app-txt 
    Issue #3176836 by marcelovani: Added some extra bits

  • marcelovani committed 12a4c5c on 3176836-app-txt 
    Issue #3176836 by marcelovani: Fixing tests

  • marcelovani committed 12a4c5c on 8.x-1.x 
    Issue #3176836 by marcelovani: Fixing tests
  • marcelovani committed 3dec611 on 8.x-1.x
    Issue #3176836 by marcelovani: Added some extra bits
  • marcelovani committed f97606b on 8.x-1.x authored by nbabb
    Issue #3176836 by nbabb, marcelovani: Add support for app-ads.txt
marcelovani’s picture

marcelovani
Primary language Portuguese, Brazil
Location London
commented
Status: Needs review » Fixed

Added some missing bits. Thanks for the patch.

Status: Fixed » Closed (fixed)

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