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

Default sequences replaced when comparing media from FPTR #545

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chxmberland
Copy link
Contributor

Linked issues

N/A

Summarize your change.

This When a user opens media from Flow Production Tracking, and then compares it to another piece of media in Flow Production Tracking, the default sequences are overwritten. Meaning, the user can no longer access the clips that they initially opened from FPTR. This can interrupt workflows.

If this change is implemented, along with another PR into OpenRV, users will have the option to enable a parameter in Flow Production Tracking / Session Prefs that will prevent this from happening.

This PR, paired with a PR in Commercial RV, will resolve this issue. The changes made in this PR include a command that modifies a global preference to prevent media from being overwritten while comparing media.

Describe the reason for the change.

User wanted to be able to compare media without overwriting the original media.

Describe what you have tested and on which operating system.

MacOS 14.5 Sonoma

chxmberland and others added 2 commits August 23, 2024 11:07
### Linked issues

N/A

Based on this review:
AcademySoftwareFoundation#530 (comment)

### Summarize your change.

Include statement in `src/lib/app/TwkApp/Bundle.cpp` has been fixed.

### Describe the reason for the change.

The include statement was trying to include a file that dosen't exist,
`file.h` instead of `File.h`.

### Describe what you have tested and on which operating system.

Builds correctly on...

- MacOS 14.5 Sonoma

Signed-off-by: Ben Chamberland <[email protected]>
@chxmberland chxmberland force-pushed the fix/default-sequence-replaced-by-comparison_SG-31769 branch from fa64748 to ed5819c Compare August 23, 2024 15:07
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