Skip to content
This repository was archived by the owner on Sep 20, 2024. It is now read-only.

Adds public functions to App object: - #6

Closed
tonybarbieri wants to merge 1 commit into
shotgunsoftware:masterfrom
Psyop:add_app_public_methods
Closed

Adds public functions to App object:#6
tonybarbieri wants to merge 1 commit into
shotgunsoftware:masterfrom
Psyop:add_app_public_methods

Conversation

@tonybarbieri

Copy link
Copy Markdown

show_save_as_dlg
show_change_version_dlg
can_save_as

These are useful for calling outside of the App itself.

Necessary for shotgunsoftware/tk-multi-snapshot#6

show_save_as_dlg
show_change_version_dlg
can_save_as
@tonybarbieri

Copy link
Copy Markdown
Author

If pull request #12 is accepted, the can_save_as public function is no longer needed.

-tony

@alandann

Copy link
Copy Markdown
Contributor

Hey Tony,

We're having issues pulling in your changes - it's probably because it's taken me a month to get to them :(

Has anything changed with your repo that might stop us having access?

Thanks
Alan

@tonybarbieri

Copy link
Copy Markdown
Author

Hrm...No nothing should have changed.

@tonybarbieri

Copy link
Copy Markdown
Author

Is it because mine is still set to private?

@alandann

Copy link
Copy Markdown
Contributor

Possibly - although Manne was able to pull some of your changes in before..

@alandann

Copy link
Copy Markdown
Contributor

...he isn't able to with these now though!

@tonybarbieri

Copy link
Copy Markdown
Author

Let me make them public. It might be because the shotgun repos went public but the forked repos havent :).

@tonybarbieri

Copy link
Copy Markdown
Author

It is now public! Does it work?

@alandann

Copy link
Copy Markdown
Contributor

It does indeed - thanks Tony - I may have some questions while I go through these......

@tonybarbieri

Copy link
Copy Markdown
Author

Please feel free to hit me up!

@alandann

alandann commented Sep 1, 2013

Copy link
Copy Markdown
Contributor

Done! - will release shortly...

Thanks again for this Tony :)
Alan

@alandann alandann closed this Sep 1, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants