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

[17.0][IMP] heldpesk*: Update logos #693

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Conversation

etobella
Copy link
Member

@etobella etobella commented Feb 4, 2025

Updating logs to the new format

@OCA-git-bot
Copy link
Contributor

Hi @victoralmau,
some modules you are maintaining are being modified, check this out!

@etobella etobella linked an issue Feb 4, 2025 that may be closed by this pull request
@etobella etobella added this to the 17.0 milestone Feb 4, 2025
@etobella etobella changed the title [IMP] heldpesk*: Update logos [17.0][IMP] heldpesk*: Update logos Feb 4, 2025
@Christian-RB
Copy link
Contributor

IMO the color is too dark, maybe we could keep it closer to the old one?

@pedrobaeza
Copy link
Member

And I think all the changes can be in the same commit, as they are all related to the same.

@etobella
Copy link
Member Author

etobella commented Feb 4, 2025

On the next migration, the changes will be splitted, for this reason I did several commits, but I can squash them 😉

@pedrobaeza
Copy link
Member

Yeah, but that will be done by the git command. Here you can see the change all together.

Copy link

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

LGTM!

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-693-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a27e96a into OCA:17.0 Feb 5, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1378e29. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Helpdesk Logo
6 participants