Conversation
* remove dead import * remove the traslation and comentary * root_text on segment leve * remove root_text legacy * remove segment_info * get segment detail * remove legacy get by id * remove the segment detail service * remove put and post partial * cleanup * fix segment fetch * drop sheet concept * drop text uploader * remove config * remove mapping * remove all * remvoe group * Revert "remvoe group" This reverts commit 72925d1. * Revert "remove all" This reverts commit c24962a. * test
#634) * refactor: remove deprecated collections endpoints and unused functionality * refactor: remove duplicate collection upload function and unused import --------- Co-authored-by: TenzDelek <tibetdelek@gmail.com>
Text Migration
Fix missing TableOfContentResponse import and migration duplicate col…
Remove v2 prefix and tags from collections and texts routers
Add v2 suffix to API router tags for collections, segments, and texts
…t detail service to return nested content with sections and segments
V 2 api integration check
…vent unbounded loops - Added constants for maximum pagination skip (10000), limit (100), and fetch iterations (100) - Sanitized user-controlled skip and limit inputs in `_fetch_matching_related_segments_by_text_id` to prevent unbounded loops - Added iteration counter to break out of fetch loop after MAX_FETCH_ITERATIONS to prevent infinite loops - Applied safe_skip and safe_limit throughout pagination logic for consistent bounds
|
- Introduced an optional `title` parameter in the `fetch_texts_by_category` function to filter results based on a case-insensitive substring of the title. - Updated related functions and views to support the new title filtering feature. - Enhanced tests to verify the correct behavior of title filtering in various scenarios, ensuring that the title parameter is correctly passed and handled. - Set coverage threshold to 80% in the CI configuration.
Segment info migration
feat(texts): add title filtering to text fetching endpoints
- Added validation for the MongoDB connection string to ensure it is configured before attempting to initialize the client. - Implemented error handling for invalid connection strings and other exceptions during client initialization, with appropriate logging. - Refactored the initialization of Beanie document models to use a constant list for better maintainability. - Updated tests to cover scenarios where the connection string is empty or invalid, ensuring robust error handling.
feat(mongo_database): enhance MongoDB connection handling in lifespan
…hing functions - Updated the `fetch_texts_by_category` and `get_texts_by_collection_from_openpecha` functions to accept `category_id` and `collection_id` as optional parameters. - Adjusted related service and view functions to handle cases where these IDs are not provided, ensuring proper functionality without them. - Enhanced tests to verify the behavior of the updated functions when no category or collection ID is supplied, confirming that the parameters are omitted as expected.
Segment info migration
feat(texts): make category_id and collection_id optional in text fetc…
Segment info migration
Search implementation
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
3 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.