Changing logic to create private channels instead of IM Groups (#276)

* Changing logic to create private channels instead of IM Groups

* Fixing some linting

Author: stefdworschak

accounts/admin.py

@@ -4,6 +4,7 @@
 from django.contrib.auth.decorators import login_required
 
 from .models import CustomUser, Organisation
+from accounts.models import SlackSiteSettings
 
 
 class CustomUserAdmin(BaseUserAdmin):
@@ -38,7 +39,7 @@ class CustomUserAdmin(BaseUserAdmin):
 
     form = UserChangeForm
     add_form = UserCreationForm
-    list_display = ('email', 'full_name', 'is_superuser', 'user_type',
+    list_display = ('email', 'username', 'full_name', 'is_superuser', 'user_type',
                     'is_external')
     list_filter = ('is_staff', 'is_superuser', 'is_active', 'groups',
                    'is_external')
@@ -52,3 +53,4 @@ class CustomUserAdmin(BaseUserAdmin):
 admin.site.login = login_required(admin.site.login)
 admin.site.register(CustomUser, CustomUserAdmin)
 admin.site.register(Organisation)
+admin.site.register(SlackSiteSettings)

accounts/migrations/0018_slacksitesettings.py

@@ -0,0 +1,27 @@
+# Generated by Django 3.1.13 on 2022-12-19 14:45
+
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('accounts', '0017_customuser_timezone'),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name='SlackSiteSettings',
+            fields=[
+                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('enable_welcome_emails', models.BooleanField(default=True)),
+                ('communication_channel_type', models.CharField(choices=[('slack_private_channel', 'Private Slack Channel'), ('other', 'Other')], default='slack_private_channel', max_length=50)),
+                ('slack_admins', models.ManyToManyField(related_name='slacksitesettings', to=settings.AUTH_USER_MODEL)),
+            ],
+            options={
+                'verbose_name': 'Slack Site Settings',
+                'verbose_name_plural': 'Slack Site Settings',
+            },
+        ),
+    ]

accounts/models.py

@@ -6,8 +6,14 @@
 from django.contrib.auth.models import AbstractUser
 
 from .lists import LMS_MODULES_CHOICES, TIMEZONE_CHOICES
+from main.models import SingletonModel
 from teams.lists import LMS_LEVELS
 
+COMMUNICATION_CHANNEL_TYPES = [
+    ('slack_private_channel', 'Private Slack Channel'),
+    ('other', 'Other'),
+]
+
 
 class UserType(Enum):
     SUPERUSER = 0
@@ -144,6 +150,13 @@ def participant_label(self):
             return 'Hackathon Enthusiast'
         else:
             return 'Hackathon Veteran'
+    
+    def is_participant(self, hackathon):
+        if not hackathon:
+            return False
+        
+        return self in hackathon.participants.all()
+
 
     @property
     def user_type(self):
@@ -180,3 +193,20 @@ def user_type(self):
             else:
                 # A non-specified group
                 return None
+
+
+class SlackSiteSettings(SingletonModel):
+    """ Model to set how the showcase should be constructed"""
+    slack_admins = models.ManyToManyField(CustomUser,
+                                          related_name="slacksitesettings")
+    enable_welcome_emails = models.BooleanField(default=True)
+    communication_channel_type = models.CharField(
+        max_length=50, choices=COMMUNICATION_CHANNEL_TYPES,
+        default='slack_private_channel')
+
+    def __str__(self):
+        return "Slack Settings"
+
+    class Meta:
+        verbose_name = 'Slack Site Settings'
+        verbose_name_plural = 'Slack Site Settings'

competencies/migrations/0003_auto_20221219_1421.py

@@ -0,0 +1,21 @@
+# Generated by Django 3.1.13 on 2022-12-19 14:21
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('competencies', '0002_auto_20220808_1029'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='competencyassessment',
+            options={'verbose_name': 'Competency Self Assessment', 'verbose_name_plural': 'Competency Self Assessments'},
+        ),
+        migrations.AlterModelOptions(
+            name='competencyassessmentrating',
+            options={'verbose_name': 'Competency Self Assessment Rating', 'verbose_name_plural': 'Competency Self Assessment Ratings'},
+        ),
+    ]

hackathon/migrations/0047_auto_20221219_1421.py

@@ -0,0 +1,18 @@
+# Generated by Django 3.1.13 on 2022-12-19 14:21
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('hackathon', '0046_auto_20220113_1350'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='hackathon',
+            name='is_public',
+            field=models.BooleanField(default=True),
+        ),
+    ]

hackathon/migrations/0048_auto_20221219_1655.py

@@ -0,0 +1,18 @@
+# Generated by Django 3.1.13 on 2022-12-19 16:55
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('hackathon', '0047_auto_20221219_1421'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='hackteam',
+            name='communication_channel',
+            field=models.CharField(blank=True, default='', help_text='Usually a link to the Private Slack Channel, but can be a link to something else.', max_length=255),
+        ),
+    ]

hackathon/models.py

@@ -188,7 +188,7 @@ class HackTeam(models.Model):
                                    on_delete=models.SET_NULL)
     communication_channel = models.CharField(
         default="", max_length=255, blank=True,
-        help_text=("Usually a link to the Slack group IM, but can be a link "
+        help_text=("Usually a link to the Private Slack Channel, but can be a link "
                    "to something else."))
 
     def __str__(self):

hackathon/templates/hackathon/hackathon_view.html

@@ -99,6 +99,9 @@
                                 <p class="hackathon-sub-details"><i class="far fa-question-circle"></i> Status: {{ hackathon.status }}</p>
                                 <p class="hackathon-sub-details"><i class="fas fa-chalkboard-teacher"></i> Organiser: {{ hackathon.organiser }}</p>
                                 <p class="hackathon-sub-details"><i class="fas fa-users"></i>
+                                    Participants: {{ hackathon.participants.all|length }} / Teams: {{ hackathon.teams.all|length }}</p>
+                                </p>
+                                <p class="hackathon-sub-details"><i class="fas fa-user-friends"></i>
                                     Max Participants: {% if hackathon.max_participants %}{{ hackathon.max_participants }}{% else %}Unlimited{% endif %}
                                     {% if hackathon.max_participants_reached %}(Max Reached){% endif %}
                                 </p>

hackathon/templates/hackathon/includes/enrollpart.html

@@ -46,7 +46,7 @@
                 <a class="ci-link ml-2" href="/teams/{{participant_team.id}}/">{{participant_team}}</a>
             </strong>
             <div class="mt-2 slack-mpim">
-                {% include 'includes/create_slack_mpim.html' with team=participant_team button_class='btn btn-sm btn-ci' %}
+                {% include 'includes/create_slack_private_channel.html' with team=participant_team button_class='btn btn-sm btn-ci' %}
             </div>
             {% else %}
             You have not been assigned a team yet.

hackathon/views.py

@@ -360,13 +360,13 @@ def view_hackathon(request, hackathon_id):
     paginator = Paginator(teams, 3)
     page = request.GET.get('page')
     paged_teams = paginator.get_page(page)
-    create_group_im = (settings.SLACK_ENABLED and settings.SLACK_BOT_TOKEN)
+    create_private_channel = (settings.SLACK_ENABLED and settings.SLACK_BOT_TOKEN)
 
     context = {
         'hackathon': hackathon,
         'teams': paged_teams,
         'change_status_form': ChangeHackathonStatusForm(instance=hackathon),
-        'create_group_im': create_group_im,
+        'create_private_channel': create_private_channel,
     }
 
     return render(request, "hackathon/hackathon_view.html", context)