Extensible display_hook

[th-jyu]

Add ability to customise the button text, menu and action names/properties using a hook instead.

Current implementation imitates existing behaviour without changes to any currently authored configs or settings

Added

• display config dict with hook and optional settings
• MultiPublish2.display_hook get-only property to expose display:hook instance

Changed

• Migrated menu name and properties setup in MultiPublish2 (app.py) into display:hook.menu_name and display:hook.menu_properties and switched to call them
• Migrated AppDialog._display_action_name logic into display:hook.action_name and switched to call it
• Switched text used for AppDialog.ui.publish to use display:hook.button_name
  • which continues to use display:hook.action_name

[codecov]

Codecov Report

❌ Patch coverage is 85.71429% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 40.41%. Comparing base (0e27293) to head (d48ed6b).

Files with missing lines	Patch %	Lines
hooks/display_hook.py	92.59%	2 Missing ⚠️
python/tk_multi_publish2/dialog.py	0.00%	2 Missing ⚠️
python/tk_multi_publish2/__init__.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage   40.18%   40.41%   +0.23%     
==========================================
  Files          61       62       +1     
  Lines        5027     5050      +23     
==========================================
+ Hits         2020     2041      +21     
- Misses       3007     3009       +2     

Flag	Coverage Δ
Linux	40.41% <85.71%> (+0.23%)	⬆️
Python-3.10	40.41% <85.71%> (+0.23%)	⬆️
Python-3.11	40.41% <85.71%> (+0.23%)	⬆️
Python-3.13	40.41% <85.71%> (+0.23%)	⬆️
Python-3.9	40.42% <85.71%> (+0.23%)	⬆️
Windows	40.41% <85.71%> (+0.23%)	⬆️
macOS	40.41% <85.71%> (+0.23%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[RicardoMusch]

This looks really usefull! :)

[th-jyu]

Thanks @RicardoMusch , for us internally it has been stable. Going to add documentations and then should be ready to merge

[th-jyu]

I've left this as v2.x.x as I'm not sure how the merge and release process goes from Autodesk's side, but this should be changed prior to release

[th-jyu]

Should I update all instances of v2_10_7 to v2_10_8 and make it match?

[th-jyu]

Should this default implementation be moved into base_hooks?