Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#521 Add Export to CSV for Queue Details and Enhance Admin Configuration #588

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hanzheg
Copy link

@hanzheg hanzheg commented Dec 6, 2024

We add a feature to export selected Queue details to a CSV file, including associated Meeting and Attendee data, improving administrative functionality.

@zqian
Copy link
Member

zqian commented Dec 9, 2024

@hanzheg The PR works for me.

The downloaded csv file contains those fields:

Queue ID, Queue Name, Status, Created At, Hosts, Allowed Backends, Description, Meeting Count, and Attendee Count

vs if one uses the current Download feature inside the /manage page, the downloaded csv file contains fields of:

queue_id,attendee_id,attendee_user_id,attendee_uniqname,attendee_last_name,attendee_first_name,host_id,host_uniqname,host_last_name,host_first_name,meeting_type,meeting_url,agenda,meeting_created_at

Can you make the admin export be consistent with same fields?

@zqian zqian linked an issue Dec 9, 2024 that may be closed by this pull request
@jonespm
Copy link
Member

jonespm commented Dec 9, 2024

Thanks for the contribution but I don't think this is really what we were looking for here.

There's an existing method that was added to the views.py for ExportMeetingStartLogs.

To fix this one for admin I'd probably do a few things that I should have mentioned on the issue.

  1. Probably refactor the database logic from that method either into the models.py MeetingStartLogsView proxy or into a separate utils file.
  2. Rewrite this to be less memory intensive, avoid a fetchall and instead have a loop that does a fetchmany
  3. Have a parameter for whether or not to do the filter by user. (To make it work for admins)
  4. Call this method from where you are defining the admin action.

I can add some more detail to the ticket, some of these might be subtasks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide a way for admins to export everything
3 participants