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

Convert actions to tk-multi-loader2 schema #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ahuge
Copy link

@Ahuge Ahuge commented Mar 11, 2021

Changing references of sg_data to sg_publish_data

This change is discussed on the community forum topic "Tk-multi-shotgunpanel & tk-multi-loader2 using same actions code?"

The gist of it is that I am aware this would require a breaking api change, however the two actions generation code only have the change of the method parameter names.
This synchronizes them and no longer requires small hacks like the ones suggested in the forum topic.

@Ahuge
Copy link
Author

Ahuge commented Mar 11, 2021

I know there are rough usage metrics on which is used more, shotgunpanel or loader2, I assumed that loader2 is used more but if I am incorrect, it may be easier to break api compat of loader2 and conform it to shotgunpanel.

Let me know if you'd prefer me to submit the inverse change on loader2 instead of here!

Changing references of `sg_data` to `sg_publish_data`
@Ahuge
Copy link
Author

Ahuge commented Mar 11, 2021

Error is with azure build system

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