Open
Conversation
There was a problem hiding this comment.
Pull Request Overview
This WIP pull request introduces paid contests and awards functionality with TON blockchain integration. The changes enable contest creators to configure entry fees and prize pools, with automated award distribution to winners using TON wallets.
Key Changes
- Integration of TON blockchain functionality for wallet creation, balance checking, and transaction processing
- Database schema updates to support payment tracking, wallet management, and contest award types
- Award distribution scheduler that automatically pays out winners after contest completion
- Service layer refactoring to consolidate business logic and improve separation of concerns
Reviewed Changes
Copilot reviewed 43 out of 45 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/ton/*.go | TON blockchain client, wallet management, and transaction operations |
| pkg/scheduler/scheduler.go | Task scheduler for periodic award distribution |
| migrations/*.sql | Database schema for payments, wallets, and award tracking |
| internal/storage/repository/postgres/*.go | Repository layer updated to support transactions and new entities |
| internal/app/service/*.go | Service layer with consolidated business logic for contests, entries, and payments |
| internal/app/distributor/distributor.go | Award distribution logic for completed contests |
| internal/app/handler/*.go | API handlers refactored to use service layer |
| internal/config/config.go | TON client configuration added |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| whereClauses = append(whereClauses, fmt.Sprintf("LOWER(title) LIKE LOWER($%d)", paramIndex)) | ||
| queryArgs = append(queryArgs, "%"+filters.Title+"%") | ||
| countArgs = append(countArgs, "%"+filters.Title+"%") | ||
| paramIndex++ |
There was a problem hiding this comment.
This definition of paramIndex is never used.
…test, entry, and submission flows
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
1 participant
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.