feat: Add field to allow auto-apply without SSO

license_manager/apps/api/serializers.py

@@ -358,6 +358,7 @@ class Meta:
             'expired_subscription_modal_messaging',
             'hyper_link_text_for_expired_modal',
             'url_for_expired_modal',
+            'enable_auto_applied_subscriptions_with_universal_link'
         ]
 
     def get_subscription_for_auto_applied_licenses(self, obj):

license_manager/apps/subscriptions/admin.py

@@ -421,6 +421,7 @@ class CustomerAgreementAdmin(admin.ModelAdmin):
         'expired_subscription_modal_messaging',
         'hyper_link_text_for_expired_modal',
         'url_for_expired_modal',
+        'enable_auto_applied_subscriptions_with_universal_link'
     )
     custom_fields = ('subscription_for_auto_applied_licenses',)
 

license_manager/apps/subscriptions/constants.py

@@ -25,6 +25,14 @@ class LicenseTypesToRenew:
     )
 
 
+class SubscriptionPlanShouldAutoApplyLicensesChoices:
+    CHOICES = (
+        (None, "----------"),
+        (True, "Yes"),
+        (False, "No")
+    )
+
+
 class SubscriptionPlanChangeReasonChoices:
     NONE = None
     NEW = "new"

license_manager/apps/subscriptions/forms.py

@@ -19,6 +19,7 @@
     MAX_NUM_LICENSES,
     MIN_NUM_LICENSES,
     SubscriptionPlanChangeReasonChoices,
+    SubscriptionPlanShouldAutoApplyLicensesChoices,
 )
 from license_manager.apps.subscriptions.models import (
     CustomerAgreement,
@@ -40,6 +41,14 @@ class SubscriptionPlanForm(forms.ModelForm):
     """
     Form used for the SubscriptionPlan admin class.
     """
+
+    should_auto_apply_licenses = forms.ChoiceField(
+        choices=SubscriptionPlanShouldAutoApplyLicensesChoices.CHOICES,
+        required=False,
+        label="Should auto apply licenses",
+        help_text="Whether licenses from this Subscription Plan should be auto applied."
+    )
+
     # Extra form field to specify the number of licenses to be associated with the subscription plan
     num_licenses = forms.IntegerField(
         label="Number of Licenses",
@@ -273,6 +282,7 @@ def populate_subscription_for_auto_applied_licenses_choices(self, instance):
         active_plans = SubscriptionPlan.objects.filter(
             customer_agreement=instance,
             is_active=True,
+            should_auto_apply_licenses=True,
             start_date__lte=now,
             expiration_date__gte=now
         )
@@ -284,7 +294,9 @@ def populate_subscription_for_auto_applied_licenses_choices(self, instance):
         choice_field = forms.ChoiceField(
             choices=choices,
             required=False,
-            initial=empty_choice if not current_plan else (current_plan.uuid, current_plan.title)
+            initial=empty_choice if not current_plan else (current_plan.uuid, current_plan.title),
+            help_text="The subscription plan with auto applied licences enabled. If you do not see the subscription in "
+            "the dropdown, please enable \"Should auto apply licenses\" from the subscription plan admin page."
         )
         self.fields['subscription_for_auto_applied_licenses'] = choice_field
 

license_manager/apps/subscriptions/migrations/0071_customeragreement_enable_auto_applied_subscriptions_with_universal_link_and_more.py

@@ -0,0 +1,23 @@
+# Generated by Django 4.2.16 on 2024-10-02 20:49
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('subscriptions', '0070_customeragreement_expired_subscription_modal_messaging_and_more'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='customeragreement',
+            name='enable_auto_applied_subscriptions_with_universal_link',
+            field=models.BooleanField(default=False, help_text='By default, auto-applied subscriptions are only granted when learners join thier enterprise via SSO Checking this box will enable subscription licenses to be applied when a learner joins the enterprise via Universal link as well'),
+        ),
+        migrations.AddField(
+            model_name='historicalcustomeragreement',
+            name='enable_auto_applied_subscriptions_with_universal_link',
+            field=models.BooleanField(default=False, help_text='By default, auto-applied subscriptions are only granted when learners join thier enterprise via SSO Checking this box will enable subscription licenses to be applied when a learner joins the enterprise via Universal link as well'),
+        ),
+    ]

license_manager/apps/subscriptions/models.py

@@ -178,6 +178,15 @@ class CustomerAgreement(TimeStampedModel):
         )
     )
 
+    enable_auto_applied_subscriptions_with_universal_link = models.BooleanField(
+        default=False,
+        help_text=_(
+            "By default, auto-applied subscriptions are only granted when learners join thier enterprise via SSO "
+            "Checking this box will enable subscription licenses to be applied when a learner joins the enterprise via "
+            "Universal link as well"
+        )
+    )
+
     history = HistoricalRecords()
 
     @property
@@ -596,6 +605,7 @@ def num_revocations_remaining(self):
 
         num_revocations_allowed = ceil(self.num_licenses * (self.revoke_max_percentage / 100))
         return num_revocations_allowed - self.num_revocations_applied
+
     num_revocations_remaining.fget.short_description = "Number of Revocations Remaining"
 
     @property
@@ -1036,6 +1046,7 @@ class License(TimeStampedModel):
     .. pii_types: id,email_address
     .. pii_retirement: local_api
     """
+
     class Meta:
         indexes = [
             models.Index(fields=["subscription_plan", "status"], name="subscription_plan_status_idx"),

pylintrc

@@ -64,7 +64,7 @@
 # SERIOUSLY.
 #
 # ------------------------------
-# Generated by edx-lint version: 5.3.6
+# Generated by edx-lint version: 5.4.0
 # ------------------------------
 [MASTER]
 ignore = ,migrations, settings, wsgi.py
@@ -401,4 +401,4 @@ int-import-graph =
 [EXCEPTIONS]
 overgeneral-exceptions = builtins.Exception
 
-# fe60e0fc0935adda7e448a1daaa3ef3547352d92
+# ea586deca5871e992466c748232382f9dfadff18