Skip to content

feat: add support for locale based publishing [TOL-2810]#1465

Merged
Chris Helgert (chrishelgert) merged 5 commits intomainfrom
feat/TOL-2810
Feb 21, 2025
Merged

feat: add support for locale based publishing [TOL-2810]#1465
Chris Helgert (chrishelgert) merged 5 commits intomainfrom
feat/TOL-2810

Conversation

@chrishelgert
Copy link
Contributor

@chrishelgert Chris Helgert (chrishelgert) commented Feb 20, 2025

Summary

Support for using locale-based publishing is added to keep the status more granular.

Kapture.2025-02-20.at.16.43.54.mp4

@chrishelgert Chris Helgert (chrishelgert) marked this pull request as ready for review February 20, 2025 15:46
Copy link
Member

@marcolink Marco Link (marcolink) left a comment

Choose a reason for hiding this comment

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

I don't fully understand the * key, but it generally looks good :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Fantastic work on this, I know quite a few devs are going to be so excited to see this land!!

The publishing steps in both entry transformation and derivation are pretty similar, extracting the logic (or at least the locale extraction part) into a shared helper might reduce duplication and simplify maintenance, but that is just a nitpick and is absolutely not necessary for this changeset.

@chrishelgert Chris Helgert (chrishelgert) merged commit 652913f into main Feb 21, 2025
6 checks passed
@chrishelgert Chris Helgert (chrishelgert) deleted the feat/TOL-2810 branch February 21, 2025 06:38
@contentful-automation
Copy link

🎉 This PR is included in version 4.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

@2wce Kudakwashe Mupeni (2wce) left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants