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

#326: enhancement: add 'rename_layers' photoshop function #698

Conversation

Guilhemz
Copy link

@Guilhemz Guilhemz commented Mar 8, 2024

Changelog Description

Add Photoshop route to rename list of layers.

Fix quadproduction/issues#326

Additional info

Each entry needs to have an 'id' corresponding to the layer id to rename and a 'new_name' which contains the new name of the layer.

Choose a reason for hiding this comment

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

le fichier a été supprimer mais le nouveau n'a pas été mis au même endroit

Copy link
Author

Choose a reason for hiding this comment

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

J'ai replacé le fichier au bon endroit, attention cependant, il faudra de toute façon regénéré entièrement le fichier d'extension en prenant aussi en compte les modifications d'Hadrien sur sa PR (#699)

Copy link
Author

Choose a reason for hiding this comment

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

Ah non pardon, le commit ne s'est pas bien fait.
En revanche, problème de mon côté, Git refuse de considérer le fichier comme nouveau et ne veut pas le pusher.
Dans la mesure où, de toute façon, il faudra générer une nouvelle version de l'extension à partir de la prochaine mise en prod, ça ne me paraît pas obligatoire de pusher le fichier

@BenSouchet
Copy link

Et le mieux serait de revert les changements car il y a plein de lignes qui 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")

@Guilhemz
Copy link
Author

Merci pour tes retours @BenSouchet ,
Concernant les changements sur le script client.js je ne sais pas pourquoi ils ont eu lieu. C'est peut-être un coup de Visual Studio Code qui aurait pu appliquer un formattage auto, mais j'ai cherché dans les préférences et il n 'y a rien de setté à ce niveau là. Je suspecte plutôt Git qui aurait pu agir sur le fichier (notamment via l'action trim trailing whitespace qui est lancée automatiquement lors d'un commit).

Pour l'instant j'ai simplement récupéré la structure du fichier de base et ajouté ma fonction uniquement via l'éditeur inclus dans Github, mais on aura probablement le soucis de nouveau à la prochaine modification du fichier.

@BenSouchet
Copy link

Merci pour tes retours @BenSouchet , Concernant les changements sur le script client.js je ne sais pas pourquoi ils ont eu lieu. C'est peut-être un coup de Visual Studio Code qui aurait pu appliquer un formattage auto, mais j'ai cherché dans les préférences et il n 'y a rien de setté à ce niveau là. Je suspecte plutôt Git qui aurait pu agir sur le fichier (notamment via l'action trim trailing whitespace qui est lancée automatiquement lors d'un commit).

Pour l'instant j'ai simplement récupéré la structure du fichier de base et ajouté ma fonction uniquement via l'éditeur inclus dans Github, mais on aura probablement le soucis de nouveau à la prochaine modification du fichier.

D'acc, pour info, il est possible de skip les hooks de git via l'ajout du flag --no-verif lors des git commit et des git push

@BenSouchet BenSouchet changed the base branch from main to release/3.16.9-quad-1.10.0 March 19, 2024 09:45
@BenSouchet BenSouchet closed this Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants