Prod merge Jan 30 #53
Open
+986
−47
Conversation
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
Fix duplicate games and fetching logic
Revert duplicate games fix
implement articles from daily sun
- Fix argument parsing to only run when script is executed directly, not when imported by gunicorn - Fix MongoDB _id field modification error in Daily Sun scraper by removing _id from upsert operations - Maintain all functionality while eliminating error messages
Move signal handlers after argument parsing to fix initialization order
Fix duplicate games
Added ticket links to game objects
Added Daily Sun Images
- Updated YoutubeVideo model to include duration attribute. - Modified CreateYoutubeVideo mutation to accept duration as an argument. - Implemented get_video_duration function to fetch video duration from YouTube API. - Updated process_video_item to include video duration in the processed data. - Adjusted YoutubeVideoType to reflect the new duration field.
Added find by date query to support advanced filters for frontend
Add duration field to YoutubeVideo model and related mutations
Refactor article date handling to use ISO 8601 format
Implement sport type extraction from article titles
Added sports type to youtube videos
Fix #49 for basketball score breakdowns
|
this looks good to me |
…tead of ISO 8601 string
AnikDey-exe
approved these changes
Jan 30, 2026
Fix date comparison to use datetime object instead of ISO 8601 string
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Overview
Updates to the scraping logic, new integrations for daily sun content, and other fixes.
Changes Made
Daily Sun Integration: Implemented scraping and display for Daily Sun articles and images.
Basketball Support: Added scraping for basketball score breakdowns and box scores.
Media Enhancements: Added ticket links to game objects and duration/sport type fields to YouTube video models.
Duplicate Prevention: Implemented logic to detect and prevent duplicate games during scraping.
Game Status: Added handling for tournaments and TBD/TBA game updates.
Filtering & Metadata: Added "find by date" query support for advanced frontend filters and standardized article dates to ISO 8601.
Server Config: Fixed Gunicorn argument parsing, initialization order, and updated the service port to 8000 (previously 8001).
Database: Resolved MongoDB
_idfield errors and removed verbose logging.Test Coverage
Next Steps (delete if not applicable)
Related PRs or Issues (delete if not applicable)
Screenshots (delete if not applicable)
Screen Shot Name