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.

Sign up for GitHub

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

Jump to bottom

1.1.0 #22

Merged
merged 22 commits into from
Oct 30, 2023
Merged

1.1.0 #22

merged 22 commits into from
Oct 30, 2023

Conversation

BenSouchet
Copy link
Contributor

@BenSouchet BenSouchet commented Oct 27, 2023 •
edited
Loading

Resolving:

CAILLOT Cyprien and others added 22 commits October 10, 2023 10:41
Save my stuff
516d920
add integrate_version_to_task.py
c7e92e3
Add documenation
b932548
@Guilhemz
#152: feat: add new create_playblast tvPaint plugin with correspondin…
3d1a67a 
…g config
@Guilhemz
Merge branch 'enhancement/152-simplify-playblasterreviewer-creator-fo…
90d08bb 
…r-tvpaint' of https://github.com/quadproduction/openpype-custom-plugins into enhancement/152-simplify-playblasterreviewer-creator-for-tvpaint
@Guilhemz
#152: bugfix: for tvpaint/create_playblast, update settings key index…
26bfc49 
… because of errors. Also remove unused settings
@Guilhemz
#152: clean: remove unused private variable
eda0625
@Guilhemz
#152: clean: remove whitespaces
ccc9663
@Guilhemz
#152: feature: add missing settings option for review enabling and ac…
b362245 
…tivation at launch
@Guilhemz
#152: bugfix: create_playblast - remove hardcoded value for active bo…
0579aa5 
…olean at subset initialization
@Guilhemz
#152: clean: remove unused imports and prints
bfffbf5
@Guilhemz
#152: clean: update class name
cea21a8
@Guilhemz
#159: feat: add new create_publish_layout tvPaint subset and integrat…
5c88a91 
…e_kitsu_sequence Kitsu module
@Guilhemz
#159: clean: create_publish_layout - update label from publish sequen…
84282ea 
…ce to review
@Guilhemz
#159: clean: update from 'ignore_layers_transparency' to 'keep_layers…
1bc11d8 
…_transparency' for better user comprehension
@Guilhemz
Merge branch 'enhancement/159-enhancement-layout-reviews-publishing-t…
fc9fe6b 
…ool' of https://github.com/quadproduction/openpype-custom-plugins into enhancement/159-enhancement-layout-reviews-publishing-tool
@Guilhemz
#159: bugfixe: update setting variable name
0d7af58
@Guilhemz
#159: bugfixe: remove keep_layers_transparency attribute initializati…
4d40395 
…on at subset creation and replace it by keep_layers_transparency
@BenSouchet
Add .idea to .gitignore
27cb0f9
@BenSouchet
Merge remote-tracking branch 'origin/enhancement/152-simplify-playbla…
4a02c84 
…sterreviewer-creator-for-tvpaint'
@BenSouchet
Merge remote-tracking branch 'origin/enhancement/159-enhancement-layo…
15ef38d 
…ut-reviews-publishing-tool'
@BenSouchet
Merge remote-tracking branch 'origin/feature/integrate_version_to_tas…
27f1fcc 
…k' into 1.1.0
@BenSouchet
Copy link
Contributor Author

Merci de regarder @Guilhemz et @ccaillot que votre code est bien dans la version et valider la review pour que je puisse faire la version lundi

@ccaillot
Copy link
Collaborator

ccaillot commented Oct 30, 2023 •
edited
Loading

Salut @BenSouchet, tu as mentionné le ticket #107 dans ton commentaire et je ne le vois pas intégrer dans la PR, on devrait avoir quad_pyblish_module/plugins/hiero/publish/integrate_version_to_task.py

@BenSouchet
Copy link
Contributor Author

Salut @BenSouchet, tu as mentionné le ticket #107 dans ton commentaire et je ne le vois pas intégrer dans la PR, on devrait avoir quad_pyblish_module/plugins/hiero/publish/integrate_version_to_task.py

Je n'avais pas push le merge de la branche en local, là cela devrait être mieux

ccaillot
ccaillot approved these changes Oct 30, 2023
View reviewed changes
Copy link
Collaborator

@ccaillot ccaillot left a comment

Choose a reason for hiding this comment

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

Rien à redire en ce qui me concerne

Guilhemz
Guilhemz approved these changes Oct 30, 2023
View reviewed changes
Copy link
Contributor

@Guilhemz Guilhemz left a comment

Choose a reason for hiding this comment

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

Bon pour moi aussi !

@BenSouchet BenSouchet merged commit ca63b99 into main Oct 30, 2023
@BenSouchet BenSouchet temporarily deployed to staging October 30, 2023 14:39 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Guilhemz Guilhemz Guilhemz approved these changes

@ccaillot ccaillot ccaillot approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BenSouchet @ccaillot @Guilhemz