Remove languages functionality#633
Draft
keeandev wants to merge 79 commits intojesus/expo-54-2026from
Draft
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Updates to Preview Branch (jesus/remove-languages) ↗︎
Tasks are run on every commit but only new migration files are pushed.
View logs for this Workflow Run ↗︎. |
…ed columns (and possibly tables) - All glory to God our Father and the Lord Jesus Christ and the Holy Spirit. - Enhanced migration system logic and user flow - Improved error handling and feedback during migrations - Simplified migration state/reload handling after migration completes - Updated and cleaned migration files and related utilities
…uilds - Added 'migration-databases/' to .gitignore to prevent tracking of migration database files. - Removed deprecated production logic for useDrizzleStudio in metro.config.js to streamline module resolution. - Updated package.json and package-lock.json to include new dependencies for improved functionality. - Refactored hooks to conditionally use development or production versions of useDrizzleStudio and useExpoDb, enhancing performance and maintainability.
- Updated .gitignore to include specific database files and scripts for better tracking. - Added new database file (1.0.db) for migration purposes. - Introduced scripts for consolidating databases, replacing databases in iOS Simulator, and restarting apps in the simulator, improving development workflow and database management.
…ipts - Updated .gitignore to include new testing client migration files and scripts for better management. - Refactored migration analysis documentation to reflect changes in database paths and added cleanup steps. - Introduced new scripts for consolidating databases, replacing databases in the iOS Simulator, and restarting apps, enhancing the testing workflow. - Added README for testing client migrations to provide clear instructions and file descriptions.
- Updated migration analysis documentation to reflect changes in script usage for both iOS and Android platforms. - Introduced unified scripts for replacing databases and restarting apps, improving the testing workflow. - Enhanced README for testing client migrations with clear instructions and platform detection details. - Added new scripts `replace-device-db.sh` and `restart-device-app.sh` to streamline database management across platforms.
- Added `replace-device-db.sh` and `restart-device-app.sh` to .gitignore for better management of testing client migrations. - Retained legacy iOS-only scripts for backwards compatibility.
- Deleted `replace-ios-sim-db.sh` and `restart-ios-sim-app.sh` scripts from the repository as they are no longer needed for migration processes. - Updated .gitignore to reflect the removal of these legacy scripts, improving clarity in the testing client migrations.
Co-authored-by: realdinozoid <realdinozoid@gmail.com>
…steps - Updated `analyze-migration-steps.md` to include critical wait time for app initialization after restart. - Modified `replace-device-db.sh` and `restart-device-app.sh` to support optional platform overrides for better flexibility in testing. - Added validation for platform overrides and improved messaging for users when both iOS and Android devices are detected. - Ensured existing WAL/SHM files are removed before copying new databases to maintain a clean state.
…dates - Updated migration scripts from versions 2.0 to 2.1 and 2.1 to 2.2 to directly manipulate JSON data in the 'data' column of raw PowerSync tables. - Added preflight checks to ensure the existence of relevant raw tables before executing migrations. - Enhanced logging for better visibility during migration processes.
- Updated `analyze-migration-steps.md` to include detailed instructions for creating test data that covers all migration scenarios. - Improved clarity on migration requirements, including legacy fields, new fields, and edge cases. - Added verification steps to ensure successful migration execution and data integrity. - Emphasized the importance of checking for data inconsistencies and provided SQL query examples for validation.
- Added retry logic for connecting to Expo devtools with exponential backoff and jitter to handle connection failures gracefully. - Introduced cleanup functions for managing connection states and handling process termination signals. - Enhanced error logging for unhandled promise rejections and uncaught exceptions to improve debugging. - Implemented a check for Expo dev server accessibility before attempting WebSocket connections to prevent unnecessary retries.
- Updated `analyze-migration-steps.md` to include critical steps for consolidating the test database after data insertion, ensuring a clean state before copying to devices. - Refactored migration scripts to utilize the `ensureTableExists` function, ensuring necessary PowerSync tables are created if they do not exist. - Improved JSON-first approach in migration scripts, emphasizing the use of `json_extract` and `json_set` for data manipulation. - Enhanced logging and preflight checks to improve migration reliability and visibility during execution.
Co-authored-by: realdinozoid <realdinozoid@gmail.com>
- Introduced a new GitHub Actions workflow for TruffleHog to scan for secrets in the codebase. - The workflow is triggered on pushes to the 'dev', 'main', and 'ci' branches.
…enesis-ai-dev/langquest into jesus/improve-migration-system
…n handling - Added PreAuthMigrationCheck to manage migrations before user authentication, improving user experience. - Updated MigrationScreen to support both pre-auth and post-auth migration flows. - Enhanced System class with a new method to check for migrations needed before authentication. - Refactored layout to integrate PreAuthMigrationCheck, ensuring migrations are handled seamlessly before login.
- Introduced a new database migration file (0.0.db) to support upcoming migration processes. - This file serves as a foundational step for managing data migrations in the application.
…d remove unused async storage dev tools from layout
…sus/remove-languages
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Functional changes (actual logic changes) 1. Removed deprecated source_language_id field Removed from ChapterData interface in publishService.ts Removed from database insert operations in: publishService.ts (2 locations) BibleAssetsView.tsx BibleRecordingView.tsx (2 locations) 2. Made languoid_id required (non-nullable) Changed languoid_id: string | null → languoid_id: string in translationContentLinks interface Removed fallback logic: c.languoid_id ?? c.source_language_id ?? null → c.languoid_id 3. Added validation guards Added checks to skip content when languoid_id is missing in: publishService.ts (line ~1790) BibleAssetsView.tsx (line ~1163) BibleRecordingView.tsx (lines ~2060 and ~2143) Formatting/typecheck fixes (cosmetic) 1. TypeScript type fixes scripts/mcp-server.ts: Removed unnecessary as unknown as NodeJS.Timeout casts (2 locations) 2. Tailwind className reordering NextGenNewTranslationModal.tsx: Reordered Tailwind classes (e.g., border-warning/30 bg-warning/5 → bg-warning/5 border-warning/30) — 7 locations 3. Import reordering BibleRecordingView.tsx: Moved RecordingHelpDialog import to match import order convention Summary: Functional changes remove the deprecated source_language_id field, make languoid_id required, and add validation. The rest are formatting/type fixes.
… implement migration for language preferences - Updated components (LanguageCombobox, LanguageSelect, etc.) to replace references to the old Language type with the new Languoid type. - Adjusted local store to manage uiLanguoid and savedLanguoid instead of uiLanguage and savedLanguage. - Implemented a migration script to convert existing language preferences in AsyncStorage from the old format to the new Languoid format. - Ensured compatibility in various views and hooks by updating state management and localization logic to reflect the new Languoid structure.
…nctionality - Replaced all instances of 'Language' with 'Languoid' across components, including LanguageSelect, LanguageCombobox, and their respective hooks. - Updated state management to reflect the new Languoid structure, ensuring compatibility in various views and hooks. - Introduced new LanguoidCombobox and LanguoidSelect components for improved languoid selection. - Adjusted migration scripts to handle languoid preferences correctly. - Enhanced localization and UI elements to align with the new terminology.
- Added orphaned view detection in createUnionViews() method - Queries for all existing union views that have corresponding _synced tables - Identifies views not in current drizzleSchema and drops them - Handles schema refactoring cases (e.g., language -> languoid migration) - Logs orphaned views found and cleanup operations performed Co-authored-by: Keean <keeandev@users.noreply.github.com>
Merges orphaned view cleanup functionality that removes views no longer defined in drizzleSchema during createUnionViews() Co-authored-by: Keean <keeandev@users.noreply.github.com>
github-actions
bot
dismissed
their stale review
Fingerprint is now compatible - no native build required
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
2 participants
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.