Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvements to the Video Selections Process #239

Merged
merged 6 commits into from
Jan 31, 2024

Conversation

fs3itz
Copy link
Contributor

@fs3itz fs3itz commented Jan 23, 2024

No description provided.

@fs3itz fs3itz marked this pull request as draft January 24, 2024 12:14
@fs3itz fs3itz force-pushed the gui/feature/video-selection-improvements branch from adae0b2 to 94a9d90 Compare January 29, 2024 03:43
@fs3itz
Copy link
Contributor Author

fs3itz commented Jan 29, 2024

rebased onto main

@fs3itz fs3itz marked this pull request as ready for review January 29, 2024 05:17
@fs3itz fs3itz requested review from a team and akriese and removed request for a team January 29, 2024 05:17
@akriese
Copy link
Contributor

akriese commented Jan 29, 2024

I think, we should merge #251 before this and then rebase this again if there are conflicts

added video format check on video selection
added fileChooserFilter to filter for certain File Extensions

Signed-off-by: Fabian Seitz <[email protected]>
Signed-off-by: Fabian Seitz <[email protected]>
@fs3itz fs3itz force-pushed the gui/feature/video-selection-improvements branch from 4648db4 to 7b06f0b Compare January 31, 2024 02:45
@fs3itz
Copy link
Contributor Author

fs3itz commented Jan 31, 2024

rebased onto main after #251 was merged

Signed-off-by: Fabian Seitz <[email protected]>

- fixed linting in lib 2

Signed-off-by: Fabian Seitz <[email protected]>

- fixed rebase issues
- moved codec checking into lib2
- changed projectManager allowed file extensions from json to mkv

Signed-off-by: Fabian Seitz <[email protected]>
@fs3itz fs3itz force-pushed the gui/feature/video-selection-improvements branch from e9de3de to 6b7aefe Compare January 31, 2024 09:26
- replaced isLosslessCodec in DifferenceGenerator with checkFile

Signed-off-by: Fabian Seitz <[email protected]>
Signed-off-by: Fabian Seitz <[email protected]>
@fs3itz fs3itz force-pushed the gui/feature/video-selection-improvements branch from 9689399 to 9d64f18 Compare January 31, 2024 09:51
@akriese akriese self-requested a review January 31, 2024 09:57
@akriese akriese merged commit f50e541 into main Jan 31, 2024
8 checks passed
@zino212 zino212 deleted the gui/feature/video-selection-improvements branch January 31, 2024 11:37
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.

2 participants