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

Photoshop: mark publishable instances #1093

Merged

Conversation

kalisp
Copy link
Member

@kalisp kalisp commented Mar 5, 2021

Closes #643

Added icons:
℗ - highlight publishable instance
▼ - highlight loaded container

Added SubsetManager and implemented methods
Switched representation from namedtuple to attr as former was immutable
Added rename_layer method
Fixed small bugs
Fixed loading of last workfile in PS AND AE

Requires:
ynput/avalon-core#294

Implemented list_instances and remove_instance methods
Warn if duplicate subsets found
Updates subset name in instance metadata and layer name
Use layer name and PUBLISH_ICON, not subset name
Changed structure of metadata from {} to []
Added rename_layer method
Switched to attr instead of namedtuple (same as in AE)
PS in Windows opens path with / wrong.
File cannot be saved as its name is full path
AE in Windows opens path with / wrong.
File cannot be saved as its name is full path
@kalisp kalisp added the 2x-dev label Mar 5, 2021
@kalisp kalisp added this to the 2.15.4 milestone Mar 5, 2021
@kalisp kalisp requested a review from a team March 5, 2021 11:59
@kalisp kalisp self-assigned this Mar 5, 2021
temp_layers_meta = {}
for layer_meta in layers_data:
layer_id = layer_meta.get("uuid") or \
(layer_meta.get("members")[0])
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line over-indented for hanging indent

if group.long_name:
for directory in group.long_name[::-1]:
name = directory.replace(stub.PUBLISH_ICON, '').\
replace(stub.LOADED_ICON, '')
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line over-indented for hanging indent

"Instance subset names are not unique. " +
"Remove duplicates via SubsetManager."
)
assert len(subset_names) == len(set(subset_names)), msg
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no newline at end of file

@mkolar mkolar changed the base branch from 2.x/main to 2.x/develop March 10, 2021 15:57
@mkolar mkolar merged commit c88400c into 2.x/develop Mar 10, 2021
@kalisp kalisp deleted the feature/643-mark-publishable-instances-in-photoshop branch March 11, 2021 12:18
@mkolar mkolar changed the title Feature/643 mark publishable instances in photoshop Photoshop: mark publishable instances Mar 12, 2021
@mkolar mkolar modified the milestones: 2.15.4, 2.16.0 Mar 18, 2021
@mkolar mkolar added the type: enhancement Enhancements to existing functionality label Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants