-
Notifications
You must be signed in to change notification settings - Fork 36
Main temp #440
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
Open
QuinnNTonic
wants to merge
209
commits into
main
Choose a base branch
from
main_temp
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Main temp #440
Conversation
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
…ic into frontend/activityfeed
…atically generated and approved. Generated and applied at: Mon Aug 18 22:46:17 UTC 2025
- Add ARCHITECTURE.md documenting monorepo structure, user roles, tech stack, and Orcha → RestClient migration pattern - Add branch-integration-plan.md tracking outstanding work across branches (activity feed, win25 features, waiver, etc.) - Update README Node LTS from gallium to jod - Remove unused EnrollmentComponent import 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Merged changes from commit 69b2b38 with conflict resolution: - Added OpenSSL legacy provider flag to README instructions - Kept Firebase emulator environment variables in npm scripts - Most .docker changes already present via prior merges 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Merged changes from commit a026599 with conflict resolution: - Kept firestore emulator reference in README - Docker and environment changes already present via prior merges 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Merged changes from commit e2a0cad with conflict resolution: - Updated environment checks to use !== 'local' pattern for: - Email verification bypass (auth, user services) - Email/SMS notification sending - User session signup flow - Kept production noreply email (noreply@involvemint.io) - Kept POI image resize logic in storage service - Kept jest-marbles testing dependency This enables local development with Firebase emulators while maintaining email verification in test and production environments. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This major architectural change replaces the Orcha RPC framework with a standard REST API approach: - Replace client-side orchestrations with RestClient services - Replace server-side orchestrations with NestJS controllers - Add repository pattern utilities for type-safe queries - Rename InvolvemintOrchestrations enum to InvolvemintRoutes - Preserve Activity Feed state management (pending migration to RestClient) Cherry-picked from ae8da81 with conflicts resolved to preserve: - Firebase emulator configuration - Activity Feed imports and routes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from f0fabfa with conflicts resolved: - Kept Firebase emulator configs and OpenSSL flag in scripts - Removed Orcha packages from package.json - Kept bad-words for profanity filtering Changes include: - E2E test migration to use RestClients instead of Orchestrations - Removed orchestration files from shared domain - Renamed orchestrations.ts to involvemint-routes.ts - API documentation updates 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from 0c49e5d. Fixed aspect ratio issues in market component thumbnails. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from e258f79. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from 7cfb328. Makes new users automatically become ChangeMakers. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from 4520866. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from d912087. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from 3664219. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from d102854. Fixed prepopulation to use backend service instead of frontend state. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from 6e72866 with conflicts resolved: - Merged isActivityFeedRoute() with markViewedNewAccount() methods - Kept environment.*.json in .gitignore - Kept HEAD's init.sql with quoted database name Features: - User state tracking for viewing new account setup - Red dot indicators for unread/new items - Database schema updates for user tracking 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cherry-picked from 4315a2f. Added red dot indicator locations in browse projects and wallet components. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove Orcha orchestrations and replace with REST API pattern for Activity Feed functionality (posts, comments, likes, flags). Client changes: - Add ActivityPostRestClient and CommentRestClient - Update effects to use REST clients instead of orchestrations - Remove orchestration files Server changes: - Add ActivityPostController and CommentController - Register controllers in ServerAPIModule - Services and repositories already compatible 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add digest method to ActivityPostRestClient - Update ActivityFeedComponent to use RestClient instead of Orchestration - Remove duplicate USE_FIRESTORE_EMULATOR import in app.module.ts Both client and server builds now pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Node 16 is based on Debian Buster which has reached end-of-life and its package repositories are no longer available. Updated to Node 18 (Debian Bookworm) which is still supported. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The EnrollmentsModule was incorrectly imported into CreateCmProfileModule (added in commit 3a75604 for waiver feature but never actually used). This caused two issues: 1. Routing conflict - EnrollmentsModule's RouterModule.forChild() routes conflicted with CreateCmProfile's routes, causing wrong page to display 2. API error - EnrollmentsComponent subscribed to enrollments$ which triggered enrollment.get() API call, failing for users without a CM The import was dead code - neither the component nor template used any waiver or enrollment functionality. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The handle slide (3rd slide) had two issues: 1. Called next() instead of submit() - there is no 4th slide 2. Checked phone.invalid instead of handle.invalid Changed button to: - Call submit() to create the ChangeMaker profile - Disable when handle is invalid, verifying, or not unique - Label changed from "Next" to "Create Profile" Also removed stray "goal" text at end of file. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
a40f0d3 to
0c52325
Compare
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.
No description provided.