Merge pull request #753 from readthedocs/davidfischer/show-publisher-…

…allowed-domains

Show the publisher's allowed domains

adserver/models.py

@@ -472,6 +472,9 @@ def daily_earn_cache_key(self):
         today = get_ad_day().date()
         return f"daily-earn::{self.slug}::{today:%Y-%m-%d}"
 
+    def allowed_domains_as_list(self):
+        return self.allowed_domains.split()
+
     def total_payout_sum(self):
         """The total amount ever paid out to this publisher."""
         total = self.payouts.filter(status=PAID).aggregate(

adserver/templates/adserver/publisher/settings.html

@@ -23,6 +23,22 @@ <h1>{% blocktrans %}Publisher settings for {{ publisher }}{% endblocktrans %}</h
     <div class="row">
 
       <div class="col-md-8 col">
+
+        {% if publisher.allowed_domains %}
+          <fieldset class="my-3 ">
+            <legend>{% trans 'Domains' %}</legend>
+            <ul>
+              {% for domain in publisher.allowed_domains_as_list %}
+                <li>{{ domain }}</li>
+              {% endfor %}
+            </ul>
+            <p class='form-text small text-muted'>
+              {% url 'support' as support_url %}
+              {% blocktrans %}These are your domains where ads may be displayed. Please <a href="{{ support_url }}?subject=New+Publisher+Domain">let us know</a> if you plan to run ads on additional domains so we may review them.{% endblocktrans %}
+            </p>
+          </fieldset>
+        {% endif %}
+
         {% crispy form form.helper %}
       </div>
 

adserver/views.py

@@ -757,7 +757,7 @@ def ignore_tracking_reason(self, request, advertisement, offer):
             )
             reason = "Mismatched browser"
         elif offer.publisher.allowed_domains and not is_allowed_domain(
-            offer.url, offer.publisher.allowed_domains.split()
+            offer.url, offer.publisher.allowed_domains_as_list()
         ):
             log.log(
                 self.log_security_level,