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

Removing old API methods #27

Merged
merged 6 commits into from
Nov 13, 2024

Conversation

jakubjezek001
Copy link
Member

@jakubjezek001 jakubjezek001 commented Nov 7, 2024

Changelog Description

Removing some old functionality after the update to new publisher workflow.

Additional info

  • pyblish callback for switching publish instance attribute
  • pype tag metadata helper functions
  • newAssetPublishing key in editorial instances.

Testing notes:

All should be working as it was before.

- Removed set_publish_attribute and get_publish_attribute functions from the codebase as they were not being used.
- Updated parameter description in the imprint function to correct spelling error and improve clarity.
- Removed unnecessary code related to setting publish attribute and added a call to set_trackitem_ayon_tag within the imprint function.
- Removed unnecessary function definitions and imports
- Refactored existing code for clarity and simplicity
@jakubjezek001 jakubjezek001 self-assigned this Nov 7, 2024
@jakubjezek001 jakubjezek001 added the type: enhancement Improvement of existing functionality or minor addition label Nov 7, 2024
Removed the deprecated "newAssetPublishing" feature from the codebase to ensure compatibility and maintain clean code structure.
- Removed deprecated track item Pype functions and their aliases from the codebase to clean up unnecessary legacy code.
Copy link
Contributor

@robin-ynput robin-ynput left a comment

Choose a reason for hiding this comment

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

Tested locally with these changes, confirmed Create, Load and Publish works as expected.

@jakubjezek001 jakubjezek001 merged commit ab733f0 into develop Nov 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants