Add a Warning pre-auth page for Zoom users #142 (#194)

.github/workflows/ci.yml

@@ -51,7 +51,7 @@ jobs:
             # Setup
             - uses: actions/setup-python@v2
               with:
-                  python-version: 3.7
+                  python-version: '3.x'
             - run: pip install pipenv
 
             # Install the stack and run the tests

Pipfile

@@ -10,7 +10,7 @@ invoke = ">=1.4"
 # https://github.com/docker/compose/issues/6839
 colorama = {version = ">=0.4, <1"}
 pypiwin32 = {version = "*", sys_platform = "== 'win32'"}
-pywin32 = {version = "*", sys_platform = "== 'win32'"}
+#pywin32 = {version = "*", sys_platform = "== 'win32'"}
 
 [requires]
 python_version = "3"

config/routes.yaml

@@ -74,3 +74,7 @@ discord_authenticate:
 zoom_authenticate:
     path:     /auth/zoom
     controller: JoliCode\SecretSanta\Controller\ZoomController::authenticate
+
+zoom_pre_auth_warning:
+    path:     /intro/zoom
+    controller: JoliCode\SecretSanta\Controller\ZoomController::preAuthWarning

src/Controller/ZoomController.php

@@ -36,6 +36,13 @@ public function __construct(string $zoomClientId, string $zoomClientSecret, Rout
         $this->zoomClientSecret = $zoomClientSecret;
     }
 
+    public function preAuthWarning()
+    {
+        return $this->render('santa/pre_auth_warning.html.twig', [
+            'title' => 'Zoom Secret Santa warning',
+        ]);
+    }
+
     /**
      * Ask for Zoom authentication and store the AccessToken in Session.
      */

templates/content/homepage.html.twig

@@ -23,7 +23,7 @@
                     <a class="action hvr-buzz action-discord" href="{{ path('run', { application: 'discord' }) }}">
                         <i class="custom-fa-discord"></i> Run on Discord
                     </a>
-                    <a class="action hvr-buzz action-zoom" href="{{ path('run', { application: 'zoom' }) }}">
+                    <a class="action hvr-buzz action-zoom" href="{{ path('zoom_pre_auth_warning') }}">
                         <i class="custom-fa-zoom"></i> Run on Zoom
                         <sup style="position:absolute;top:8px;right:4px;font-size: 0.6em;">beta</sup>
                     </a>

templates/santa/pre_auth_warning.html.twig

@@ -0,0 +1,35 @@
+{% extends 'base.html.twig' %}
+
+{% block content %}
+    <div class="content error-page">
+        <div class="is-center">
+            <h2>⚠️ Zoom support may not be what you expect ⚠️</h2>
+
+            <p>
+                This application can only show and send private messages to <strong>your users</strong>. That means the users you
+                can see as a Zoom Admin here: <a href="https://zoom.us/account/user">https://zoom.us/account/user</a>.
+            </p>
+
+            <p>
+                We cannot show you your contacts, this is a known issue of the Zoom API and you
+                can found <a href="https://github.com/jolicode/secret-santa/issues/142#issuecomment-781204731">more information here</a>.
+            </p>
+        </div>
+    </div>
+    <div class="jumbotron" style="padding: 20px 0;">
+        <div class="jumbotron-inner">
+            <a class="action hvr-buzz action-zoom" href="{{ path('run', { application: 'zoom' }) }}">
+                <i class="custom-fa-zoom"></i> Run on Zoom anyway!
+                <sup style="position:absolute;top:8px;right:4px;font-size: 0.6em;">beta</sup>
+            </a>
+        </div>
+    </div>
+    <div class="content error-page">
+        <div class="is-center">
+            <p>
+                Sorry for any disappointment!<br>
+                <em>(Also, this application is open-source, you can help!)</em>
+            </p>
+        </div>
+    </div>
+{% endblock content %}