Added the functionality to record a GCP Billing Id attached to an event in the event model (#2375)

This feature allows addition of the gcp billing id to events for a more
streamlined workflow of attaching shared billing ids to the cloud
identities of the users participating in the events. This addresses the
t-cairem [issue
#18](T-CAIREM#18)

The following key changes have been implemented:
- Added a `gcp_billing_id` field in the events model & the migration for
it.
- Added a conditional rendering of Billing Accounts to choose from if
the user has a cloud identity (only HDN users have cloud identities
currently)
- Implemented signals in the hdn-research-environment package to handle
the changes in the EventApplication & CloudIdentity.

Another part to be implemented alongside for the feature to work is :
Bumping the hdn-research-environment-package(#2377).

Author: Rutvikrj26

physionet-django/events/apps.py

@@ -1,5 +1,10 @@
 from django.apps import AppConfig
+from physionet.settings.base import ENABLE_CLOUD_RESEARCH_ENVIRONMENTS
 
 
 class EventsConfig(AppConfig):
     name = 'events'
+
+    def ready(self):
+        if ENABLE_CLOUD_RESEARCH_ENVIRONMENTS:
+            import environment.signals

physionet-django/events/fixtures/demo-events.json

@@ -33,4 +33,4 @@
         "allowed_domains": null
     }
   }
-]
+]

physionet-django/events/forms.py

@@ -1,4 +1,5 @@
 from django import forms
+import logging
 
 from events.widgets import DatePickerInput
 from events.models import Event, EventApplication, EventAgreement, EventDataset, CohostInvitation
@@ -10,6 +11,8 @@
     (0, 'Decline')
 )
 
+logger = logging.getLogger(__name__)
+
 
 class AddEventForm(forms.ModelForm):
     """
@@ -30,6 +33,29 @@ def __init__(self, user, *args, **kwargs):
         self.host = user
         super(AddEventForm, self).__init__(*args, **kwargs)
 
+        # Add billing account choices if user has cloud identity
+        if user and hasattr(user, 'cloud_identity'):
+            self.fields['gcp_billing_id'] = forms.ChoiceField(
+                required=False,
+                label='Billing Account',
+                help_text='Select a billing account to associate with this event.',
+                choices=[('', 'No billing account')]
+            )
+            # Set initial value if updating an existing event
+            if self.instance and self.instance.pk and self.instance.gcp_billing_id:
+                self.initial['gcp_billing_id'] = self.instance.gcp_billing_id
+
+            try:
+                # making a local import only if needed to avoid import issues in testing
+                from environment.services import get_billing_accounts_list
+                billing_accounts = get_billing_accounts_list(user)
+                billing_choices = [('', 'No billing account')]
+                for account in billing_accounts:
+                    billing_choices.append((account['id'], f"{account['name']} ({account['id']})"))
+                self.fields['gcp_billing_id'].choices = billing_choices
+            except Exception:
+                logger.info("Unable to get billing accounts list.")
+
     def clean(self):
         cleaned_data = super(AddEventForm, self).clean()
         if Event.objects.filter(title=cleaned_data['title'], host=self.host).exists():
@@ -44,16 +70,21 @@ def clean(self):
     def save(self):
         # Handle updating the event
         if self.initial:
+            if 'gcp_billing_id' in self.cleaned_data:
+                self.instance.gcp_billing_id = self.cleaned_data['gcp_billing_id']
             self.instance.save()
         else:
-            Event.objects.create(title=self.cleaned_data['title'],
+            event = Event.objects.create(title=self.cleaned_data['title'],
                                  category=self.cleaned_data['category'],
                                  host=self.host,
                                  description=self.cleaned_data['description'],
                                  start_date=self.cleaned_data['start_date'],
                                  end_date=self.cleaned_data['end_date'],
                                  allowed_domains=self.cleaned_data['allowed_domains']
                                  )
+            if 'gcp_billing_id' in self.cleaned_data:
+                event.gcp_billing_id = self.cleaned_data['gcp_billing_id']
+                event.save()
 
 
 class EventApplicationResponseForm(forms.ModelForm):

physionet-django/events/migrations/0011_event_gcp_billing_id.py

@@ -0,0 +1,18 @@
+# Generated by Django 4.2.18 on 2025-03-14 19:40
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("events", "0010_alter_event_options"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="event",
+            name="gcp_billing_id",
+            field=models.CharField(blank=True, max_length=100, null=True),
+        ),
+    ]

physionet-django/events/models.py

@@ -23,6 +23,7 @@ class Event(models.Model):
     start_date = models.DateField(default=timezone.now)
     end_date = models.DateField(default=timezone.now)
     slug = models.SlugField(unique=True)
+    gcp_billing_id = models.CharField(max_length=100, blank=True, null=True)
     allowed_domains = models.CharField(blank=True, null=True, validators=[
                                        validators.validate_domain_list], max_length=100)
 

physionet-django/events/templates/events/event_detail.html

@@ -64,6 +64,10 @@ <h4>Event Details</h4>
     </div>
     <hr>
 
+    {% if event.gcp_billing_id and event.host == user or user in event.get_cohosts %}
+        <p><strong>Billing Account:</strong> {{ event.gcp_billing_id }}</p>
+    {% endif %}
+
     {% include "events/event_participation.html" %}
     <hr>
     {% if event_datasets|length != 0 %}

physionet-django/events/templates/events/event_home.html

@@ -120,8 +120,7 @@ <h5 class="modal-title">{{ info.title }}</h5>
           </div>
         </div>
     {% endfor %}
-  {% endfor %}
-   
+  {% endfor %}   
 
 <!-- Past events -->
 

physionet-django/events/views.py

@@ -133,7 +133,7 @@ def event_home(request):
 
             event_application = form.save(commit=False)
             event = event_application.event
-            # if user is not a host or a participant with cohort status, they don't have permission to accept/reject
+            # if user is not a host or a participant with cohost status, they don't have permission to accept/reject
             if not (
                 event.host == user
                 or EventParticipant.objects.filter(