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

#327: enhancement: Creer un validator pour les layers dans Photoshop #699

Conversation

hfarre
Copy link

@hfarre hfarre commented Mar 11, 2024

Fix quadproduction/issues#327

Changelog Description

Ajout d'un validator pour checker et réparer les layer soit bien en mode de fusion normal et les groupes transfert dans photoshop.

Testing notes:

  1. Dans un psd, mettre des layers dans des modes de fusions fantaisistes
  2. Tenter de publier, et constater le validator dans le subset Context

openpype/hosts/photoshop/api/extension/host/index.jsx Outdated Show resolved Hide resolved

Choose a reason for hiding this comment

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

Le ficiher a été supprimé mais regénéré dans un autre emplacement, pas dans le même dossier, à voir si c'est normal

Copy link
Author

Choose a reason for hiding this comment

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

Yes, c'est juste que je l'avais rangé au mauvais endroit à la base

@BenSouchet
Copy link

Plein de lignes ont été reformatées alors que pas nécessaire (au pire se reformatage devrait être fait dans un commit séparé, mais pas avec des modifs "logic"). Selon moi ces changements liés au lintage ne devrait pas être commit, voir avec @Guilhemz qui a le même problème dans sa PR : #698

@hfarre
Copy link
Author

hfarre commented Mar 13, 2024

Plein de lignes ont été reformatées alors que pas nécessaire (au pire se reformatage devrait être fait dans un commit séparé, mais pas avec des modifs "logic"). Selon moi ces changements liés au lintage ne devrait pas être commit, voir avec @Guilhemz qui a le même problème dans sa PR : #698

Je vais voir si @Guilhemz a trouvé d'où vient ce soucis, et je le règlerai en fonction

@BenSouchet BenSouchet changed the base branch from main to release/3.16.9-quad-1.10.0 March 19, 2024 09:44
@BenSouchet BenSouchet merged commit c6c412f into release/3.16.9-quad-1.10.0 Mar 19, 2024
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.

2 participants