Skip to content

Conversation

@loadez
Copy link
Contributor

@loadez loadez commented Oct 22, 2025

πŸ“ Description

  • Add members download button on people page
  • Implement export members as csv endpoint
  • Add export members test

βœ… Checklist

  • I have tested this change
  • This change requires documentation update

@github-project-automation github-project-automation bot moved this to Backlog in Roadmap Oct 22, 2025
@loadez loadez self-assigned this Oct 22, 2025
@skipi skipi requested a review from Copilot October 22, 2025 10:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds functionality to export organization members as a CSV file. The changes introduce a download button on the people page that allows users to export member data (name, email, and GitHub login) in CSV format.

Key changes:

  • Added a new /people/export endpoint that generates and downloads a CSV file containing organization members
  • Added a download button in the members list UI (shown only when viewing organization scope)
  • Added basic test coverage for the export endpoint

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
front/lib/front_web/controllers/people_controller.ex Implements organization_users action to fetch members and generate CSV export
front/lib/front_web/router.ex Adds GET route /people/export for the new export functionality
front/lib/front_web/templates/people/members/members_list.html.eex Adds download button in UI that links to the export endpoint
front/test/front_web/controllers/people_controller_test.exs Adds test case verifying 404 response when user lacks organization access

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@loadez loadez force-pushed the feature/add-members-download-button branch 5 times, most recently from 17c6ac7 to b7081a6 Compare October 22, 2025 17:26
@loadez loadez marked this pull request as ready for review October 22, 2025 17:31
@loadez loadez force-pushed the feature/add-members-download-button branch from b7081a6 to 8c8c231 Compare October 23, 2025 10:48
@loadez loadez force-pushed the feature/add-members-download-button branch from 8c8c231 to a380768 Compare October 24, 2025 13:30
@skipi skipi self-requested a review October 27, 2025 09:42
@loadez loadez force-pushed the feature/add-members-download-button branch 4 times, most recently from 9ecd607 to 0b24f0d Compare October 27, 2025 11:33
- Add members download button on people page
- Implement export members as csv endpoint
- Add export members test
- Add service account list download button
- Implement export endpoint at ServiceAccountController
- Add org scope check on download button
@loadez loadez force-pushed the feature/add-members-download-button branch from 0b24f0d to d64568f Compare October 27, 2025 11:41
@loadez loadez merged commit 77001c1 into main Oct 27, 2025
2 checks passed
@loadez loadez deleted the feature/add-members-download-button branch October 27, 2025 11:54
VeljkoMaksimovic pushed a commit that referenced this pull request Oct 27, 2025
## πŸ“ Description
- Add members download button on people page
- Implement export members as csv endpoint
- Add export members test

## βœ… Checklist
- [x] I have tested this change
- [ ] This change requires documentation update

Signed-off-by: Veljko Maksimovic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants