Skip to content

FEATURE: Add post language on creating a new post #33160

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

Merged
merged 16 commits into from
Jun 11, 2025

Conversation

keegangeorge
Copy link
Member

This is a second attempt at: #33001

We had to revert the commit because it was performing a site-setting check at boot time, which is prone to issues and not allowed.

This PR:

  • re-introduces the changes in the original PR
  • a fix by not performing a site-setting check at boot time (verified by: SKIP_DB_AND_REDIS=1 DISCOURSE_DEV_DB="nonexist" bin/rails runner "puts 'booted'" locally and should be caught by the new CI check introduced here: DEV: Add SKIP_DB_AND_REDIS bootability check to CI #33158)
  • adds a fix to the translation editor to not show the original post locale in the dropdown, as well as adding an indicator of what the original post locale is in a small badge in the header:
    • Screenshot 2025-06-11 at 08 42 36

@keegangeorge keegangeorge merged commit 4d380a2 into main Jun 11, 2025
16 checks passed
@keegangeorge keegangeorge deleted the post-translation-on-create branch June 11, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants