Vary message on unclaimed package for anon

Smooths out the experience a tad for proto-givers.
gratipay · Jun 15, 2017 · bf38cdb · bf38cdb
1 parent b81084e
commit bf38cdb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/tests/py/test_www_npm_package.py b/tests/py/test_www_npm_package.py
@@ -12,7 +12,7 @@ def setUp(self):
 
     def test_anon_gets_signin_page_from_unclaimed(self):
         body = self.client.GET('/on/npm/foo/').body
-        assert 'foo</a> npm package:' in body
+        assert 'foo</a> npm package on Gratipay:' in body
 
     def test_auth_gets_send_confirmation_page_from_unclaimed(self):
         self.make_participant('bob', claimed_time='now')

diff --git a/www/on/npm/%package/index.html.spt b/www/on/npm/%package/index.html.spt
@@ -51,17 +51,22 @@ if user.participant:
 <p class="sorry important-thing-at-the-top">{{ _("No description available.") }}</p>
 {% endif %}
 
-<p class="instructions">
-    {{ _( 'Apply to accept payments for the {package_link} npm package:'
-        , package_link=('<a href="' + package.remote_human_url + '">' + package_name + '</a>')|safe
-         ) }}
-</p>
-
 {% if user.ANON %}
+    <p class="instructions">
+        {{ _( 'Claim the {package_link} npm package on Gratipay:'
+            , package_link=('<a href="' + package.remote_human_url + '">' + package_name + '</a>')|safe
+             ) }}
+    </p>
     <div class="important-button">
         {{ sign_in_using(button_class='large') }}
     </div>
 {% else %}
+    <p class="instructions">
+        {{ _( 'Apply to accept payments for the {package_link} npm package:'
+            , package_link=('<a href="' + package.remote_human_url + '">' + package_name + '</a>')|safe
+             ) }}
+    </p>
+
     {% if len(emails) == 0 %}
     <p class="sorry">{{ _("No email addresses on file.") }}</p>
     {% else %}