Conversation
…ild setup, and package metadata - Added package.json with project metadata, dependencies, and build scripts. - Created tsconfig.json for TypeScript compiler options and project structure. - Configured tsup for building the library with multiple entry points and output formats.
…es to use ECClassResponse
…alities and response types
- Implemented drive API with folder permissions, info retrieval, subfolder listing, folder creation, and sharing functionalities. - Added mail API for sending single and batch emails, uploading templates, and retrieving template metadata and previews. - Updated existing hooks and entities to accommodate new features and ensure proper data handling. - Refactored registration and user APIs to streamline functionality and improve code organization.
…ities and simplifying query functions
…ovider functions feat: add organizer application API with entities, hooks, and provider functions feat: enhance photo management API with pagination and upload features feat: update sponsor API with batch operations and file uploads
…es, hooks, and provider functions
There was a problem hiding this comment.
Pull request overview
This pull request initializes a new React SDK package (@hackpsu/react-sdk) that provides shared hooks, providers, and API clients for HackPSU projects. The SDK is built with TypeScript, uses tsup for bundling, and supports both CommonJS and ES module formats. It includes Firebase authentication, TanStack Query for data fetching, and modular API clients for various HackPSU services.
Key changes:
- Sets up package configuration with proper exports for multiple entry points
- Configures TypeScript with strict mode and React JSX support
- Implements Firebase authentication provider with session management
- Creates modular API structure with 20+ service modules (events, users, teams, sponsors, etc.)
Reviewed changes
Copilot reviewed 103 out of 105 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/package.json | Package metadata with dependencies and build configuration for the React SDK |
| sdk/tsconfig.json | TypeScript compiler options for the SDK with strict mode enabled |
| sdk/tsup.config.ts | Build configuration with multiple entry points for tree-shakeable exports |
| tsconfig.build.json | Updated root build config to exclude the new sdk directory |
| sdk/index.ts | Main entry point with documentation and re-exports |
| sdk/config/* | Firebase and environment configuration modules |
| sdk/context/* | React providers for Firebase auth, layout, and auth guards |
| sdk/api/* | 20+ API service modules with hooks, providers, and entity types |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+9
to
+16
| import { | ||
| SendMailRequest, | ||
| SendBatchMailRequest, | ||
| UploadTemplateRequest, | ||
| TemplateMetadata, | ||
| PreviewMailRequest, | ||
| PreviewMailResponse, | ||
| } from "./entity"; |
There was a problem hiding this comment.
Unused import PreviewMailResponse.
| "use client"; | ||
| import { QueryClient, QueryClientProvider } from "@tanstack/react-query"; | ||
| import { FirebaseProvider } from "./FirebaseProvider"; | ||
| import { auth } from "../config"; |
There was a problem hiding this comment.
Unused import auth.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
with TypeScript configuration, buld setup, and package metadata