#727 add a command to add all pipx binary directories to PATH.

docs/source/development/getting-started.rst

@@ -19,6 +19,7 @@ to install ``nox`` in its own isolated environment.
 .. code-block:: console
 
    $ python -m pip install pipx
+   $ python -m pipx ensurepath
    $ pipx install nox
 
 You will need to have a working Redis server on your system. You may

junction/base/admin.py

@@ -23,7 +23,10 @@ class TimeAuditAdmin(admin.ModelAdmin):
 
 
 class AuditAdmin(TimeAuditAdmin):
-    list_display = ("created_by", "modified_by",) + TimeAuditAdmin.list_display
+    list_display = (
+        "created_by",
+        "modified_by",
+    ) + TimeAuditAdmin.list_display
     exclude = (
         "created_by",
         "modified_by",

junction/base/models.py

@@ -6,8 +6,7 @@
 
 
 class TimeAuditModel(models.Model):
-    """To track when the record was created and last modified
-    """
+    """To track when the record was created and last modified"""
 
     created_at = models.DateTimeField(auto_now_add=True, verbose_name="Created At")
     modified_at = models.DateTimeField(auto_now=True, verbose_name="Last Modified At")
@@ -17,8 +16,7 @@ class Meta:
 
 
 class UserAuditModel(models.Model):
-    """ To track who created and last modified the record
-    """
+    """To track who created and last modified the record"""
 
     created_by = models.ForeignKey(
         User,

junction/conferences/migrations/0001_initial.py

@@ -123,7 +123,8 @@ class Migration(migrations.Migration):
                 (
                     "conference",
                     models.ForeignKey(
-                        to="conferences.Conference", on_delete=models.deletion.CASCADE,
+                        to="conferences.Conference",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
                 (
@@ -140,7 +141,8 @@ class Migration(migrations.Migration):
                 (
                     "moderator",
                     models.ForeignKey(
-                        to=settings.AUTH_USER_MODEL, on_delete=models.deletion.CASCADE,
+                        to=settings.AUTH_USER_MODEL,
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
                 (
@@ -187,7 +189,8 @@ class Migration(migrations.Migration):
                 (
                     "conference",
                     models.ForeignKey(
-                        to="conferences.Conference", on_delete=models.deletion.CASCADE,
+                        to="conferences.Conference",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
                 (
@@ -215,7 +218,8 @@ class Migration(migrations.Migration):
                 (
                     "reviewer",
                     models.ForeignKey(
-                        to=settings.AUTH_USER_MODEL, on_delete=models.deletion.CASCADE,
+                        to=settings.AUTH_USER_MODEL,
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
             ],

junction/conferences/migrations/0005_emailreviewernotificationsetting.py

@@ -77,7 +77,8 @@ class Migration(migrations.Migration):
                 (
                     "proposal_type",
                     models.ForeignKey(
-                        to="proposals.ProposalType", on_delete=models.deletion.CASCADE,
+                        to="proposals.ProposalType",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
             ],

junction/conferences/migrations/0006_auto_20150216_1929.py

@@ -12,19 +12,26 @@ class Migration(migrations.Migration):
 
     operations = [
         migrations.RemoveField(
-            model_name="emailreviewernotificationsetting", name="conference_reviewer",
+            model_name="emailreviewernotificationsetting",
+            name="conference_reviewer",
         ),
         migrations.RemoveField(
-            model_name="emailreviewernotificationsetting", name="created_by",
+            model_name="emailreviewernotificationsetting",
+            name="created_by",
         ),
         migrations.RemoveField(
-            model_name="emailreviewernotificationsetting", name="modified_by",
+            model_name="emailreviewernotificationsetting",
+            name="modified_by",
         ),
         migrations.RemoveField(
-            model_name="emailreviewernotificationsetting", name="proposal_section",
+            model_name="emailreviewernotificationsetting",
+            name="proposal_section",
         ),
         migrations.RemoveField(
-            model_name="emailreviewernotificationsetting", name="proposal_type",
+            model_name="emailreviewernotificationsetting",
+            name="proposal_type",
+        ),
+        migrations.DeleteModel(
+            name="EmailReviewerNotificationSetting",
         ),
-        migrations.DeleteModel(name="EmailReviewerNotificationSetting",),
     ]

junction/conferences/migrations/0014_conferencesettings.py

@@ -8,8 +8,7 @@
 
 
 def add_default_values(apps, schema_editor):
-    """Add all default values
-    """
+    """Add all default values"""
     ConferenceSetting = apps.get_model("conferences", "ConferenceSetting")
     public_voting = ConferenceSettingConstants.ALLOW_PUBLIC_VOTING_ON_PROPOSALS
     display_propsals = ConferenceSettingConstants.DISPLAY_PROPOSALS_IN_PUBLIC
@@ -80,7 +79,8 @@ class Migration(migrations.Migration):
                 (
                     "conference",
                     models.ForeignKey(
-                        to="conferences.Conference", on_delete=models.deletion.CASCADE,
+                        to="conferences.Conference",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
                 (

junction/conferences/models.py

@@ -112,8 +112,7 @@ def save(self, *args, **kwargs):
         super(Conference, self).save(*args, **kwargs)
 
     def is_accepting_proposals(self):
-        """Check if any one of the proposal section is accepting proposal.
-        """
+        """Check if any one of the proposal section is accepting proposal."""
         if (
             self.status == ConferenceStatus.CLOSED_CFP
             or self.status == ConferenceStatus.SCHEDULE_PUBLISHED

junction/conferences/permissions.py

@@ -4,8 +4,7 @@
 
 
 def is_reviewer(user, conference):
-    """Returns a boolean indicating if a given user is a conference reviewer.
-    """
+    """Returns a boolean indicating if a given user is a conference reviewer."""
     if not user.is_authenticated():
         return False
 

junction/feedback/migrations/0001_initial.py

@@ -45,7 +45,8 @@ class Migration(migrations.Migration):
                 (
                     "conference",
                     models.ForeignKey(
-                        to="conferences.Conference", on_delete=models.deletion.CASCADE,
+                        to="conferences.Conference",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
                 (
@@ -142,7 +143,8 @@ class Migration(migrations.Migration):
                 (
                     "schedule_item",
                     models.ForeignKey(
-                        to="schedule.ScheduleItem", on_delete=models.deletion.CASCADE,
+                        to="schedule.ScheduleItem",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
             ],
@@ -217,7 +219,8 @@ class Migration(migrations.Migration):
                 (
                     "conference",
                     models.ForeignKey(
-                        to="conferences.Conference", on_delete=models.deletion.CASCADE,
+                        to="conferences.Conference",
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
                 (
@@ -235,7 +238,8 @@ class Migration(migrations.Migration):
             model_name="scheduleitemtextfeedback",
             name="question",
             field=models.ForeignKey(
-                to="feedback.TextFeedbackQuestion", on_delete=models.deletion.CASCADE,
+                to="feedback.TextFeedbackQuestion",
+                on_delete=models.deletion.CASCADE,
             ),
             preserve_default=True,
         ),

junction/feedback/models.py

@@ -22,8 +22,7 @@ class Meta:
 
 @python_2_unicode_compatible
 class TextFeedbackQuestion(BaseSessionQuestionMixin, TimeAuditModel):
-    """Store details about text feedback type information.
-    """
+    """Store details about text feedback type information."""
 
     title = models.CharField(max_length=255, verbose_name="Text Feedback Title")
 
@@ -44,8 +43,7 @@ def to_response(self):
 
 @python_2_unicode_compatible
 class ChoiceFeedbackQuestion(BaseSessionQuestionMixin, TimeAuditModel):
-    """Store details about text feedback type information.
-    """
+    """Store details about text feedback type information."""
 
     title = models.CharField(max_length=255, verbose_name="Choice Feedback Title")
 
@@ -71,8 +69,7 @@ def to_response(self):
 
 @python_2_unicode_compatible
 class ChoiceFeedbackQuestionValue(TimeAuditModel):
-    """Store allowed values for each choice based question
-    """
+    """Store allowed values for each choice based question"""
 
     question = models.ForeignKey(
         ChoiceFeedbackQuestion, related_name="allowed_values", on_delete=models.CASCADE

junction/feedback/service.py

@@ -58,8 +58,7 @@ def get_choice_feedback_questions(conference_id):
 
 
 def has_submitted(feedback, device_uuid):
-    """
-    """
+    """"""
     device = Device.objects.get(uuid=device_uuid)
     text_feedback = ScheduleItemTextFeedback.objects.filter(
         schedule_item_id=feedback.validated_data["schedule_item_id"], device=device

junction/feedback/views.py

@@ -48,8 +48,7 @@ def post(self, request):
 
 
 def view_feedback(request, schedule_item_id):
-    """Show text/choice feedback for the schedule.
-    """
+    """Show text/choice feedback for the schedule."""
     schedule_item = get_object_or_404(ScheduleItem, pk=schedule_item_id)
     if not can_view_feedback(user=request.user, schedule_item=schedule_item):
         return HttpResponseForbidden("Access Denied")

junction/profiles/migrations/0001_initial.py

@@ -61,7 +61,8 @@ class Migration(migrations.Migration):
                 (
                     "user",
                     models.OneToOneField(
-                        to=settings.AUTH_USER_MODEL, on_delete=models.deletion.CASCADE,
+                        to=settings.AUTH_USER_MODEL,
+                        on_delete=models.deletion.CASCADE,
                     ),
                 ),
             ],

junction/proposals/dashboard.py

@@ -197,7 +197,10 @@ def reviewer_votes_dashboard(request, conference_slug):
 
     proposal_sections = conference.proposal_sections.all()
     proposals_qs = Proposal.objects.select_related(
-        "proposal_type", "proposal_section", "conference", "author",
+        "proposal_type",
+        "proposal_section",
+        "conference",
+        "author",
     ).filter(conference=conference, status=ProposalStatus.PUBLIC)
 
     proposals = []
@@ -247,7 +250,10 @@ def second_phase_voting(request, conference_slug):
 
     proposal_sections = conference.proposal_sections.all()
     proposals_qs = Proposal.objects.select_related(
-        "proposal_type", "proposal_section", "conference", "author",
+        "proposal_type",
+        "proposal_section",
+        "conference",
+        "author",
     ).filter(conference=conference, review_status=ProposalReviewStatus.SELECTED)
 
     proposals = []
@@ -298,7 +304,10 @@ def export_reviewer_votes(request, conference_slug):
 
     proposal_sections = conference.proposal_sections.all()
     proposals_qs = Proposal.objects.select_related(
-        "proposal_type", "proposal_section", "conference", "author",
+        "proposal_type",
+        "proposal_section",
+        "conference",
+        "author",
     ).filter(conference=conference, status=ProposalStatus.PUBLIC)
     proposals_qs = sorted(
         proposals_qs, key=lambda x: x.get_reviewer_votes_sum(), reverse=True
@@ -334,7 +343,8 @@ def export_reviewer_votes(request, conference_slug):
                     [
                         comment.comment
                         for comment in p.proposalcomment_set.filter(
-                            vote=True, deleted=False,
+                            vote=True,
+                            deleted=False,
                         )
                     ]
                 )
@@ -346,7 +356,10 @@ def export_reviewer_votes(request, conference_slug):
                         p.get_reviewer_votes_count(),
                     )
                     + vote_details
-                    + (p.get_votes_count(), vote_comment,)
+                    + (
+                        p.get_votes_count(),
+                        vote_comment,
+                    )
                 )
                 if (
                     p.get_reviewer_votes_count_by_value(