Skip to content

Develop#621

Merged
ShaneIsrael merged 4 commits into
mainfrom
develop
May 2, 2026
Merged

Develop#621
ShaneIsrael merged 4 commits into
mainfrom
develop

Conversation

@ShaneIsrael
Copy link
Copy Markdown
Owner

This pull request introduces a database migration and supporting code changes to ensure that each video can only be linked to a particular game once, removing any existing duplicate links. It also updates the video player's UI by customizing the overlay gradient and cleans up related CSS. The version is bumped to 1.6.9 to reflect these updates.

Database integrity improvements:

  • Adds a unique constraint to the video_game_link table on the combination of video_id and game_id, ensuring no duplicate links between a video and a game. Existing duplicates are removed in the migration (migrations/versions/n9i0j1k2l3m4_unique_video_game_link.py, app/server/fireshare/models.py). [1] [2]
  • Updates the video scanning logic to check for an existing link before creating a new VideoGameLink, preventing duplicate entries at the application level (app/server/fireshare/cli.py).

Video player UI enhancements:

  • Adds a new CSS file, videoSkinOverrides.css, to constrain the video player's controls overlay gradient to the bottom 25% of the video, improving visual clarity (app/client/src/components/player/videoSkinOverrides.css, app/client/src/components/player/VideoJSPlayer.js). [1] [2]
  • Removes obsolete CSS that previously hid the overlay gradient (app/client/src/index.css).

Version update:

  • Bumps the application version from 1.6.8 to 1.6.9 in package.json and package-lock.json to reflect these changes (app/client/package.json, app/client/package-lock.json). [1] [2]

@ShaneIsrael ShaneIsrael merged commit 0e1d7fb into main May 2, 2026
8 checks passed
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.

1 participant