refactor: rename 'isPublished' to 'isPublic' across the codebase for …#362
Open
dothinh115 wants to merge 8 commits into
Open
refactor: rename 'isPublished' to 'isPublic' across the codebase for …#362dothinh115 wants to merge 8 commits into
dothinh115 wants to merge 8 commits into
Conversation
…clarity and consistency
…ry and update tests for accuracy
…rvice and update tests for enforced context
…ata migration tests - Introduced a new Yarn configuration file (.yarnrc.yml) to manage package settings. - Implemented a Node version check script (check-node-version.js) to ensure compatibility with Node 24.x. - Created comprehensive tests for the MetadataMigrationService, covering both SQL and MongoDB scenarios, including handling of core table overlaps, renaming logic, and idempotency of migrations. - Added functionality to probe SQL and MongoDB connections for testing purposes. - Developed mock implementations for SQL and MongoDB to facilitate unit testing of the migration service.
…ith custom fields - Added support for merging rows based on specified keys during SQL and MongoDB table renames. - Implemented logic to preserve legacy fields when conflicts arise during renaming operations. - Introduced methods to backfill missing values from legacy rows into canonical rows. - Updated tests to validate the new behavior for both SQL and MongoDB migrations. - Enhanced utility functions for physical migrations to ensure safe renaming of fields.
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.
…clarity and consistency