Skip to content

[FRI] v1.4.2_rdo7 to v1.9.1_rdo (Real branch) (RP-246) (RP-243)#12

Merged
JeanChristopheMorinRodeoFX merged 13 commits intomaster_rdofrom
dev_update_app_3
Mar 18, 2016
Merged

[FRI] v1.4.2_rdo7 to v1.9.1_rdo (Real branch) (RP-246) (RP-243)#12
JeanChristopheMorinRodeoFX merged 13 commits intomaster_rdofrom
dev_update_app_3

Conversation

@JeanChristopheMorinRodeoFX

PR #11 was not meant to be merged in master_rdo. So Created a new branch from master_rdo, then cherry-picked the commits from #10 and #11 to get all my changes. Should now be good.

meld /mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2_1.9.1/ /mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2/
meld /mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2_1.4.2/ /rodeo/repositories/tank/studio/install/apps/git/tk-multi-loader2/v1.4.2_rdo7/

That will allow you to see a diff of v1.9.1 <> v1.9.1_rdo1 and v1.4.2 <> v1.4.2_rdo3.
First meld instance will allow you to see rdo cahnges in the new app, and the second meld instance wil allow you to see rdo changes in v1.4.2.

So you can see if our changes are the same in the new app than in the old app, without seeing new code from Shotgun.

@review-ninja
Copy link

ReviewNinja

@JeanChristopheMorinRodeoFX
Copy link
Author

There is an error with this branch:

// Error: Shotgun: An exception was raised from Toolkit

__init__() takes exactly 3 arguments (2 given)
The current environment is shot_and_asset.

Code Traceback:
  File "/rdo/rodeo/repositories/tank/_sandbox/v0.16/studio/install/engines/app_store/tk-maya/v0.5.4/python/tk_maya/menu_generation.py", line 318, in _execute_within_exception_trap
    self.callback()
  File "/mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2/app.py", line 32, in <lambda>
    cb = lambda : tk_multi_loader.show_dialog(self)
  File "/mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2/python/tk_multi_loader/__init__.py", line 42, in show_dialog
    w = app.engine.show_dialog(ui_title, app, AppDialog, action_manager)
  File "/rdo/rodeo/repositories/tank/_sandbox/v0.16/studio/install/core/python/tank/platform/engine.py", line 946, in show_dialog
    dialog, widget = self._create_dialog_with_widget(title, bundle, widget_class, *args, **kwargs)
  File "/rdo/rodeo/repositories/tank/_sandbox/v0.16/studio/install/core/python/tank/platform/engine.py", line 847, in _create_dialog_with_widget
    widget = self._create_widget(widget_class, *args, **kwargs)
  File "/rdo/rodeo/repositories/tank/_sandbox/v0.16/studio/install/core/python/tank/platform/engine.py", line 825, in _create_widget
    widget = derived_widget_class(*args, **kwargs)
  File "/mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2/python/tk_multi_loader/dialog.py", line 281, in __init__
    self._add_rdo_publish_search(hlayout)
  File "/mnt/users/jcmorin/rdoenv/tank/apps/tk-multi-loader2/python/tk_multi_loader/dialog.py", line 1248, in _add_rdo_publish_search
    search = SearchWidget(self)

@JeanChristopheMorinRodeoFX
Copy link
Author

Error fixed!

small_text = "<span style='color:#2C93E2'>%s</span> by %s at %s" % (pub_type_str,
author_str,
date_str)
small_text += "<br><b>Description:</b> %s" % sg_data.get("description") or "No description given"
Copy link

Choose a reason for hiding this comment

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

Looks like some parenthesis are missing here, I'm seeing some "Description: None"

Should be
small_text += "<br><b>Description:</b> %s" % (sg_data.get("description") or "No description given")

Choose a reason for hiding this comment

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

Fixed

@kookiCT
Copy link

kookiCT commented Mar 18, 2016

👍

@kookiCT
Copy link

kookiCT commented Mar 18, 2016

Julien says 👍 too

JeanChristopheMorinRodeoFX added a commit that referenced this pull request Mar 18, 2016
[FRI] v1.4.2_rdo7 to v1.9.1_rdo (Real branch) (RP-246) (RP-243)
@JeanChristopheMorinRodeoFX JeanChristopheMorinRodeoFX merged commit 0064aab into master_rdo Mar 18, 2016
vfleuryrodeofx pushed a commit that referenced this pull request Apr 22, 2022
Introduces a new list mode in the loader which is a more compact
representation of publishes. The delegate widgets have also been moved
into the loader so that it no longer depends on the standard widgets in
the qtwidgets framework. This is a tweak to make it easier for the
loader to switch to the v3 qtwidgets framework when that is released as
part of the file manager changes.

Closes #12
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.

3 participants