Skip to content

Conversation

@jtourkos
Copy link
Contributor

@jtourkos jtourkos commented Sep 1, 2025

No description provided.

@jtourkos jtourkos requested a review from Copilot September 1, 2025 10:48
@railway-app
Copy link

railway-app bot commented Sep 1, 2025

🚅 Deployed to the search-sync-pr-10 environment in Drips App

Service Status Web Updated (UTC)
Search Sync ◻️ Removed (View Logs) Sep 1, 2025 at 12:48 pm

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 refactors database queries to align with the latest database schema changes. The changes primarily involve column name updates from camelCase to snake_case conventions and removal of the description field from search functionality.

Key changes:

  • Updates column names to use snake_case format (e.g., ownerAddressowner_address)
  • Removes description field from project search configuration and type definitions
  • Updates verification status enum values to include a new 'pending_metadata' state

Reviewed Changes

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

File Description
src/synchronizer/meilisearch/createMeiliSearchSynchronizer.ts Removes description field from search document mapping and search configuration
src/synchronizer/meilisearch/changeDetection/types.ts Updates Project type to remove description field and expand verification status enum
src/synchronizer/meilisearch/changeDetection/createPollingChangeDetection.ts Updates SQL queries to use new snake_case column names and table names

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jtourkos jtourkos requested a review from efstajas September 1, 2025 10:53
@railway-app railway-app bot temporarily deployed to Search Sync (Drips App / search-sync-pr-10) September 1, 2025 10:55 Destroyed
@railway-app railway-app bot temporarily deployed to search-sync (Drips App / staging) September 1, 2025 12:42 Inactive
@jtourkos jtourkos merged commit a9da53b into main Sep 1, 2025
4 checks passed
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / app-pr-1605) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / app-pr-1606) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot temporarily deployed to search-sync (Drips App / mainnet) September 1, 2025 12:47 Inactive
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / events-processor-pr-68) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / app-pr-1586) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / graphql-api-pr-79) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / graphql-api-pr-73) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot temporarily deployed to Search Sync (Drips App / localtestnet) September 1, 2025 12:47 Inactive
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / events-processor-pr-72) September 1, 2025 12:47 Abandoned
@railway-app railway-app bot requested a deployment to Search Sync (Drips App / app-pr-1495) September 1, 2025 12:47 Abandoned
@jtourkos jtourkos deleted the update-queries-to-latest-db-schema branch September 1, 2025 12:47
@railway-app railway-app bot temporarily deployed to search-sync (Drips App / staging) September 1, 2025 12:49 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