Skip to content

Submit strings for translation - #9377

Open
CrisBarreiro wants to merge 27 commits into
developfrom
feature/cris/cpm-prompt/request-l10n
Open

Submit strings for translation#9377
CrisBarreiro wants to merge 27 commits into
developfrom
feature/cris/cpm-prompt/request-l10n

Conversation

@CrisBarreiro

@CrisBarreiro CrisBarreiro commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Task/Issue URL: https://app.asana.com/1/137249556945/project/1212227266948491/task/1217068049663232?focus=true
Tech Design URL (if applicable):
API Proposals URL(s) (if applicable):

Description

  • Update autoconsent settings strings
  • Submir new popup strings for translation

Steps to test this PR

n/a

UI changes

image

Stack created with GitHub Stacks CLIGive Feedback 💬


Note

Low Risk
String-only changes with no behavior or logic modifications; risk is limited to copy/translation quality in the UI.

Overview
Updates Cookie Pop-Up Protection copy across locales: settings toggles now say reject optional cookies and accept cookies that can't be rejected (replacing auto-manage / pop-ups without opt-out wording), with matching description text.

Adds translated strings for a new autoconsent onboarding popup (titles and body for when auto-reject is on vs off, default/reject options, confirm button, and a settings path hint).

Also tweaks input_screen_search_only_hint in Estonian, Croatian, Romanian, and Swedish Duck.ai input strings.

Reviewed by Cursor Bugbot for commit ee113c4. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread autoconsent/autoconsent-impl/src/main/res/values/strings-autoconsent.xml Outdated
@CrisBarreiro
CrisBarreiro force-pushed the feature/cris/cpm-prompt/request-l10n branch 2 times, most recently from 42ce4c0 to 36eace9 Compare August 3, 2026 14:52

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 69035eb. Configure here.

<string name="autoconsentPopupDefaultOptionWithFeatureDisabled">"Wenn möglich ablehnen oder für mich akzeptieren"</string>
<string name="autoconsentPopupRejectOptionWithFeatureDisabled">"Ich kümmere mich weiter selbst darum"</string>
<string name="autoconsentPopupConfirmButton">"Bestätigen"</string>
<string name="autoconsentPopupSettingsHint" instruction="Bold part is a navigation path in the app: the Settings screen and the Cookie Pop-Up Protection section within it. Keep the &gt; separator."><![CDATA[Du kannst das jederzeit unter <b>Einstellungen > Cookie-Pop-up-Schutz</b>anpassen.]]></string>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing space before "anpassen" in German settings hint

Medium Severity

In the autoconsentPopupSettingsHint German string, there is no space between </b> and anpassen, producing …Cookie-Pop-up-Schutz</b>anpassen. which will render as "Cookie-Pop-up-Schutzanpassen." — two words concatenated without a space. A space is needed before "anpassen" so the sentence reads correctly.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 69035eb. Configure here.

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