Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

[2.1] [Maintenance] adjust translation guide in github issue template config #18404

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: 2.1
Choose a base branch
Loading
from
Open

[2.1] [Maintenance] adjust translation guide in github issue template config #18404

wants to merge 1 commit into from
+3 −3

Conversation

crydotsnake
Copy link
Contributor

@crydotsnake crydotsnake commented Sep 30, 2025 •
edited by coderabbitai bot
Loading

Q A
Branch? 2.1
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes #X, partially #Y, mentioned in #Z
License MIT
image

I noticed that the URL translate.sylius.com wasnt working anymore. So i adjusted the title, url, and description with the latest informations on how to contribute translations to sylius.

Summary by CodeRabbit

  • Documentation
    • Updated the issue template to replace the Translations contact with a UI Translations contact, including a refreshed link and clearer guidance for requesting or discussing UI translation updates.
    • Improves discoverability and accuracy for contributors seeking translation support.
    • No impact on application behavior or user interface.

@crydotsnake crydotsnake requested review from a team as code owners September 30, 2025 17:31
@probot-autolabeler probot-autolabeler bot added the Maintenance CI configurations, READMEs, releases, etc. label Sep 30, 2025
@coderabbitai coderabbitai
Copy link

coderabbitai bot commented Sep 30, 2025 •
edited
Loading

Walkthrough

Updated the GitHub issue template config to replace the 🇺🇸 Translations contact link with a 🌍 UI Translations contact link, including a new URL and description. No other structural or behavioral changes.

Changes

Cohort / File(s) Summary
Issue template config
.github/ISSUE_TEMPLATE/config.yml		 Replaced the 🇺🇸 Translations contact link with 🌍 UI Translations; updated URL and description; no structural or behavioral impact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I nudge the links with whiskered care,
From flags to globes—now translations share.
A hop, a tweak, no code to fear,
Just clearer paths for helpers here.
Thump-thump! The UI’s set just right—
Carrots up for cleaner light. 🌍🐇

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Title Check ✅ Passed The title clearly indicates the adjustment of the translation guide in the GitHub issue template configuration by highlighting the key file and nature of the change, making it concise, specific, and directly aligned with the pull request’s main objective.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@crydotsnake crydotsnake changed the title TASK: adjust translation guide in github issue template config [2.1] [Maintenance] adjust translation guide in github issue template config Sep 30, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 30, 2025 •
edited
Loading

❗ Preview Environment deployment failed on Bunnyshell

See: Environment Details | Pipeline Logs

Check https://github.com/Sylius/Sylius/actions/runs/18138384288 for details.

Available commands:

  • 🚀 /bns:deploy to redeploy the environment
  • ❌ /bns:delete to remove the environment

coderabbitai[bot]
coderabbitai bot reviewed Sep 30, 2025
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
.github/ISSUE_TEMPLATE/config.yml (1)

12-14: Simplify the UI Translations description to avoid URL duplication.

The about field currently repeats the URL already set on line 13. Align it with other links by mentioning the guide text first and placing the URL once in backticks:

-      about: Have a look at `https://docs.sylius.com/the-book/contributing/contributing-translations` to read more about how you can contribute translations to Sylius.
+      about: To contribute translations to Sylius, see the guide at `https://docs.sylius.com/the-book/contributing/contributing-translations`
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 926a92f and 6e1d1cd.

📒 Files selected for processing (1)
  • .github/ISSUE_TEMPLATE/config.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Static checks / PHP 8.4, Symfony ^7.3
  • GitHub Check: Static checks / PHP 8.3, Symfony ^7.3
  • GitHub Check: Static checks / PHP 8.2, Symfony ^6.4

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

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Maintenance CI configurations, READMEs, releases, etc.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@crydotsnake
Morty Proxy This is a proxified and sanitized view of the page, visit original site.