Skip to content

New Crowdin updates#3153

Merged
feruzm merged 230 commits intodevelopmentfrom
l10n_development
Mar 26, 2026
Merged

New Crowdin updates#3153
feruzm merged 230 commits intodevelopmentfrom
l10n_development

Conversation

@feruzm
Copy link
Copy Markdown
Member

@feruzm feruzm commented Mar 25, 2026

Summary by CodeRabbit

  • New Features

    • Added "Account updated" and "Weekly earnings" notifications; updated payout notification wording.
    • Added AI Assist hint: "Write your content first, then use AI Assist to polish and improve it."
  • Settings

    • Added Image Server selection with Default / Alternative / No‑CDN options.
  • Localization

    • Added/updated translations for the new notifications, AI Assist hint, and image‑server labels across locales.

@feruzm
Copy link
Copy Markdown
Member Author

feruzm commented Mar 26, 2026

@CodeRabbit how to make sure you never review l10n_development branch/PRs from that branch?

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 26, 2026

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

@feruzm, let me fetch your current CodeRabbit configuration and the available schema to give you the most accurate guidance.


🛠 Debug Information Gathered


@feruzm, there's no direct "ignore source branch" filter in CodeRabbit's configuration, but there are two reliable ways to skip all Crowdin PRs. The easiest is using ignore_title_keywords, since all Crowdin PRs are titled "New Crowdin updates".

Option 1 — .coderabbit.yaml (recommended)

Add or update .coderabbit.yaml in your repo root:

reviews:
  auto_review:
    ignore_title_keywords:
      - "Crowdin"

Any PR with "Crowdin" in the title will be automatically skipped.

Option 2 — CodeRabbit UI

Go to Settings → Reviews → Auto Review and add Crowdin to the "Ignore Title Keywords" field.


Option 3 — ignore_usernames (if Crowdin uses a bot account)

If the Crowdin PRs are authored by a dedicated bot account (e.g., crowdin-bot), you can also skip by username:

reviews:
  auto_review:
    ignore_usernames:
      - "crowdin-bot"  # replace with the actual Crowdin bot username

The ignore_title_keywords approach (Option 1) is the most straightforward for your case since all these PRs share a consistent title pattern.


✏️ Learnings added
Learnt from: feruzm
URL: https://github.com/ecency/ecency-mobile/pull/3153

Timestamp: 2026-03-26T12:30:36.579Z
Learning: In the ecency/ecency-mobile repository, to skip automatic CodeRabbit reviews for Crowdin localization PRs (from the `l10n_development` branch, titled "New Crowdin updates"), add `ignore_title_keywords: ["Crowdin"]` under `reviews.auto_review` in `.coderabbit.yaml`. There is no direct source branch filter in CodeRabbit's configuration schema.

Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.


🧠 Learnings used
Learnt from: feruzm
Repo: ecency/ecency-mobile PR: 0
File: :0-0
Timestamp: 2026-03-04T14:59:58.127Z
Learning: In the ecency/ecency-mobile repository, never review any PR that originates from the `l10n_development` branch. These are automated Crowdin localization update PRs (typically titled "New Crowdin updates") and should be skipped entirely. The maintainer (feruzm) has explicitly requested this.

Learnt from: feruzm
Repo: ecency/ecency-mobile PR: 0
File: :0-0
Timestamp: 2026-03-04T14:59:58.127Z
Learning: In the ecency/ecency-mobile repository, never review any PR that originates from the `l10n_development` branch. These are automated Crowdin localization update PRs and should be skipped entirely.

Learnt from: feruzm
Repo: ecency/ecency-mobile PR: 3110
File: src/config/locales/en-US.json:494-499
Timestamp: 2026-02-08T17:40:31.229Z
Learning: In the ecency-mobile repository, only the en-US.json locale file should be modified when adding or updating translation keys. All other locale files (e.g., es-ES.json, de-DE.json, etc.) are automatically synchronized via Crowdin integration, so they should not be manually edited in pull requests.

@feruzm feruzm merged commit bc0a506 into development Mar 26, 2026
8 checks passed
@feruzm feruzm deleted the l10n_development branch March 26, 2026 12:33
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.

1 participant