Skip to content

Conversation

@AnandajithS
Copy link
Contributor

This PR introduce the 'is_alumni' attribute to the 'Member' object to keep track of members who have graduated

  • Added a view called active_members which filters out alumni by selecting rows where 'is_alumni' is set to false

  • Updated the queries accordingly to make use of `active_members'

Closes #155

@hrideshmg hrideshmg requested a review from Copilot November 11, 2025 12:32
Copilot finished reviewing on behalf of hrideshmg November 11, 2025 12:34
Copy link
Contributor

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 alumni tracking functionality to the Member model by introducing an is_alumni boolean attribute and creating a database view to filter active members.

  • Added is_alumni boolean field to the Member model with a default value of false
  • Created active_members database view that filters out alumni (where is_alumni = false)
  • Updated all member-related queries to use the active_members view instead of querying the Member table directly

Reviewed Changes

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

Show a summary per file
File Description
migrations/20251103160042_add_alumni_attribute.sql Adds the is_alumni column to the Member table and creates the active_members view
src/models/member.rs Adds is_alumni field to Member struct and UpdateMemberInput struct
src/graphql/mutations/member_mutations.rs Updates INSERT and UPDATE queries to include the is_alumni field
src/graphql/queries/member_queries.rs Updates member queries to use active_members view instead of Member table
src/graphql/mutations/status_mutations.rs Updates status mutation query to use active_members view
src/daily_task/mod.rs Updates daily task query to use active_members view
src/database_seeder/seed.sql Updates seed data to include is_alumni values (alternating true/false pattern)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Add alumni flag and related functionality

1 participant