feat(logservice): Add possible data field to filter

[mbauer404]

Adds a possible data field to find_last_log() to enable filtering by that field if and only if it is included.

[dashed]

Suggested change

	data: dict[str, str] | None,
	data: dict[str, str] | None = None,

We can default data to be None so that we don't need to update the log_service.find_last_log() calls to add the data parameter in both sentry and getsentry.

[corps]

Yup. When changing parameters of hybrid cloud services, everything new must have default parameters due to fragmented deployment.

[corps]

what about

            last_entry_q: QuerySet[AuditLogEntry] = AuditLogEntry.objects.filter(
                organization_id=organization_id,
                target_object=target_object_id,
                event=event,
            )

            if data:
              last_entry_q = last_entry_q.filter(data=data)
            last_entry: AuditLogEntry | None = last_entry_q.last()

[dashed]

Suggested change

	data: dict[str, str] | None,
	data: dict[str, str] | None = None,

[mbauer404]

With the additional default added, I haven't been able to figure out how to get the params to include the data field when calling cls._get_abstract_rpc_methods() here. Any ideas?

[corps]

You also need to change the signature of the abstract base class (LogService) to match. When the signatures don't match it isn't included.

[corps]

= None

[dashed]

Looks good to me.