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

4.1.2 #55

Merged
merged 2 commits into from
Oct 26, 2024
Merged

4.1.2 #55

merged 2 commits into from
Oct 26, 2024

Conversation

TheTaylorLee
Copy link
Owner

@TheTaylorLee TheTaylorLee commented Oct 26, 2024

The pull request checklist must be reviewed and completed prior to completing a pull request. This is to ensure stable production images and avoid overwrites of ghcr version tags.

Pull Request Checklist

  • Reviewed the Contributing Guidelines
  • Update changelog.md
  • Update version with new semver
  • Build dev images and test changes
    • Rename a file and ensure the database entry updates paths & retains statistics.
    • Delete a file. The associated table entry should mark fileexists false and not modify the comment.
      • Update-Processed once run should mark the comment NULL and next process run should mark fileexists false.
        • To test immediately, wait for processing to pause then delete lastrun from updateprocessed log table in the sqlite db.
        • Then open an interactive shell to the container docker exec -it <container_id_or_name> pwsh
        • Run . /docker-transcodeautomation/scripts/Update-Processed.ps1
    • Replace a file with a file of the same name, but the comment should not match dta-*. The table entry should be updated and the file transcoded. (Only updates comment after update-processed run and transcoded on next processign loop.)
    • Add a new unique file and see that a table entry is created and the file is transcoded
    • Move a file to a new parent directory provided to MEDIA(MOVIE/SHOWS)FOLDERS environment variable. Statistics should be preserved.
  • If this is a new major release with breaking changes, a github release must first be created.

@TheTaylorLee TheTaylorLee self-assigned this Oct 26, 2024
@TheTaylorLee TheTaylorLee merged commit 9e510f5 into main Oct 26, 2024
1 check passed
@TheTaylorLee TheTaylorLee deleted the 4.1.2 branch October 26, 2024 19:14
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