Skip to content

Conversation

@chiptus
Copy link
Owner

@chiptus chiptus commented Aug 11, 2025

  • delete group
  • redirect to group after creation
  • add members from group view
  • refactor groups view to smaller components

@chiptus chiptus requested a review from Copilot August 11, 2025 14:42
@vercel
Copy link

vercel bot commented Aug 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
upline ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2025 2:42pm

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 implements several bug fixes and improvements for the groups functionality, including fixing group deletion policies, enabling case-insensitive email lookups, and refactoring the groups interface into smaller, more manageable components.

  • Fixes database policies to allow group creators to delete their groups
  • Implements case-insensitive email/username lookup for better user experience
  • Refactors the monolithic Groups page into multiple focused components
  • Adds direct member addition functionality from the group detail view

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
supabase/migrations/20250811120000_fix_group_deletion_policy.sql Adds missing DELETE policy for groups table
supabase/migrations/20250811130000_fix_case_insensitive_email_lookup.sql Creates case-insensitive email lookup function
src/services/groupService.ts Updates email matching to use case-insensitive queries
src/pages/Groups.tsx Refactors into smaller components and simplifies logic
src/pages/GroupDetail.tsx Adds member addition form and reorganizes code
src/hooks/useGroups.ts Removes unused functions and converts to function declarations
src/hooks/queries/useGroupsQuery.ts Adds new invite mutation and converts to function declarations
src/components/Groups/* New component files extracted from Groups page
CLAUDE.md Updates coding guidelines for component extraction

@chiptus chiptus merged commit 21bbbad into develop Aug 11, 2025
7 checks passed
@chiptus chiptus deleted the fix/group-deletion-bug branch August 11, 2025 19:18
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.

2 participants