Skip to content

Conversation

@jtourkos
Copy link
Contributor

No description provided.

@jtourkos jtourkos requested a review from Copilot August 12, 2025 09:12
@railway-app
Copy link

railway-app bot commented Aug 12, 2025

🚅 Environment sprinkler-pr-14 in Drips App has no services deployed.

@jtourkos jtourkos changed the base branch from main to splinke-to-sub-projects August 12, 2025 09:12
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 adds support for ORCID accounts by implementing a new processing pipeline alongside the existing drip lists and projects processing. The changes introduce ORCID account validation, contract interaction capabilities, and database schema updates to support the new functionality.

  • Adds ORCID account processing with validation logic to ensure accounts split 100% to their owners
  • Introduces repo sub-account driver contract integration for calculating sub-project account IDs
  • Updates database queries to use new snake_case table naming convention

Reviewed Changes

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

Show a summary per file
File Description
src/types.ts Adds repoSubAccountDriver contract address to Network type
src/queries/getTokens.ts Updates table names from PascalCase to snake_case
src/queries/getCurrentSplitsReceivers.ts Simplifies splits receiver query logic and updates schema
src/queries/getAllProjectsSortedByCreationDate.ts Adds sub-project calculation and renames function
src/queries/getAllOrcidAccountsWithSplits.ts New query for retrieving ORCID accounts with splits data
src/queries/getAllDripListsSortedByCreationDate.ts Updates table name to snake_case
src/index.ts Integrates ORCID processing into main workflow
src/getNetwork.ts Adds repoSubAccountDriver contract addresses
src/contracts/unwrapEthersResult.ts Utility for unwrapping ethers contract results
src/contracts/repoSubAccountDriverAbi.ts ABI definition for repo sub-account driver contract
src/contracts/repoSubAccountDriver.ts Contract interaction wrapper
src/contracts/drips-client.ts Refactors to use shared unwrap utility

@jtourkos jtourkos requested review from efstajas and removed request for Copilot August 12, 2025 09:14
@railway-app railway-app bot temporarily deployed to Optimistic Sprinkler (Drips App / mainnet) September 1, 2025 11:35 Inactive
@railway-app railway-app bot temporarily deployed to Metis Sprinkler (Drips App / mainnet) September 1, 2025 11:35 Inactive
@railway-app railway-app bot requested a deployment to Mainnet Sprinkler (Drips App / mainnet) September 1, 2025 11:35 Abandoned
@railway-app railway-app bot temporarily deployed to Filecoin Sprinkler (Drips App / mainnet) September 1, 2025 11:35 Inactive
@railway-app railway-app bot temporarily deployed to Optimistic Sprinkler (Drips App / mainnet) September 1, 2025 11:37 Inactive
@railway-app railway-app bot temporarily deployed to Metis Sprinkler (Drips App / mainnet) September 1, 2025 11:37 Inactive
@railway-app railway-app bot temporarily deployed to Filecoin Sprinkler (Drips App / mainnet) September 1, 2025 11:37 Inactive
@railway-app railway-app bot requested a deployment to Mainnet Sprinkler (Drips App / mainnet) September 1, 2025 11:37 Abandoned
@railway-app railway-app bot temporarily deployed to Mainnet Sprinkler (Drips App / mainnet) September 1, 2025 11:46 Inactive
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.

3 participants