Skip to content

Conversation

fer-correa
Copy link
Contributor

Ref: #517

In addition to updating the images, this PR proposes changing the way we work with images to avoid changing N language files in the future if it is necessary to change images.

If this proposal doesn't seem like a good idea, we can close the PR and continue with the image exchange following the previous pattern.

fer-correa added 2 commits May 8, 2025 23:23
 - Replace old images hosted on git with images versioned with the project

Signed-off-by: Fernando Eugenio Correa <[email protected]>
@fer-correa fer-correa requested a review from a team as a code owner May 9, 2025 03:35
Copy link

netlify bot commented May 9, 2025

Deploy Preview for ospobook ready!

Name Link
🔨 Latest commit 5436598
🔍 Latest deploy log https://app.netlify.com/sites/ospobook/deploys/681d78001748280008dd16ca
😎 Deploy Preview https://deploy-preview-615--ospobook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented May 9, 2025

Deploy Preview for ospomindmap canceled.

Name Link
🔨 Latest commit 5436598
🔍 Latest deploy log https://app.netlify.com/sites/ospomindmap/deploys/681d7800d8c3590008960a72

Copy link
Member

@anajsana anajsana left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for working on this!

@anajsana anajsana merged commit 0536cd4 into todogroup:main May 9, 2025
6 checks passed
@fer-correa fer-correa deleted the feature/ospobook-img-issue-517 branch May 11, 2025 21:14
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