feat: Updated Chat Trial Audit gating consistency

[rijuma]

Description

Ticket: COSMO-612 🔒

There was some differences between the Xpert Chat Trial conditions for the trial on the frontend vs the backend.

Removed deprecated enabled and history endpoints that were integrated in the new summary endpoint. (Ticket: COSMO-600 🔒)

These changes would make the checks more consistent.

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
learning_assistant
api.py
urls.py
views.py
Project Total

This report was generated by python-coverage-comment-action

[michaelroytman]

Looks good! I left a little feedback.

Two more things.

1. I don't think that chore is the appropriate commit type here. This is a feat, in my opinion, because it changes the functionality of the application. If you agree, could you update the commit message and PR title?
2. Could you add COSMO-600 to your pull request description and drop a link to this PR on the ticket in Jira, please? It will help us track our work better.

Thanks!

[rijuma]

Looks good! I left a little feedback.

Two more things.

1. I don't think that chore is the appropriate commit type here. This is a feat, in my opinion, because it changes the functionality of the application. If you agree, could you update the commit message and PR title?
2. Could you add COSMO-600 to your pull request description and drop a link to this PR on the ticket in Jira, please? It will help us track our work better.

Thanks!

Thank you for your exhaustive review @michaelroytman. Let me know what you think on the updates.

[michaelroytman]

Looks great! Thank you for incorporating my feedback.

I left one comment on the change I originally suggested to the >= end condition, but it's not blocking.

[michaelroytman]

I noticed that you removed the end >= today. Do you plan to make the same change on the frontend?

(By the way, I do think this is very edge-casey, and on further reflection, isn't a big deal. So feel free to change it back if it makes your life easier.)

[rijuma]

Just rolled it back to match the frontend. To be honest, since both dates includes time it's just pointless to consider the exact moment that both dates are matching. I've rolled it back to be consistent with the frontend.
Giving the extreme edge case, I don't think it's worth the surgical analysis we are putting into.