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

Nv 3526 promoting a translation group default locale not promoted #5316

Conversation

ainouzgali
Copy link
Contributor

@ainouzgali ainouzgali commented Mar 19, 2024

What change does this PR introduce?

When creating a new group -> the default locale must also be created for that group. There are 2 different changes in the changes table - 1 for the group and one for the translation. But, when promoting the translation group change, the change of adding that default locale translation is not promoted. Meaning that in production there is a situation of a translation group with no translation for the default locale

Enterprise PR

Why was this change needed?

Other information (Screenshots)

Copy link

linear bot commented Mar 19, 2024

Copy link

netlify bot commented Mar 19, 2024

Deploy Preview for dev-web-novu ready!

Name Link
🔨 Latest commit f16e735
🔍 Latest deploy log https://app.netlify.com/sites/dev-web-novu/deploys/65faeda3c9bc7c0008b332fc
😎 Deploy Preview https://deploy-preview-5316--dev-web-novu.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.

…ranslation-group-default-locale-not-promoted

# Conflicts:
#	.source
@ainouzgali ainouzgali marked this pull request as draft March 19, 2024 11:04
@ainouzgali ainouzgali marked this pull request as ready for review March 19, 2024 11:53
@LetItRock LetItRock self-requested a review March 20, 2024 09:19
…ranslation-group-default-locale-not-promoted

# Conflicts:
#	.source
…ranslation-group-default-locale-not-promoted
@ainouzgali ainouzgali merged commit 038d3f2 into next Mar 20, 2024
24 checks passed
@ainouzgali ainouzgali deleted the nv-3526-promoting-a-translation-group-default-locale-not-promoted branch March 20, 2024 14:45
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.

2 participants