Clarify registration terminology: "applied" vs "registered" - #429
Open
Strehk wants to merge 1 commit into
Open
Conversation
… registration Replaces the "angemeldet"/"registered" registration-status wording with "beworben"/"applied" (especially for supervisors) so it is clear that applicants do not hold a secured seat until the applications have been sighted and distributed. Also aligns the German status labels with the already-"applied" English equivalents and reworks the supervisor-facing copy accordingly. Login, team-tender sign-up and mailing-list wording are intentionally left unchanged. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018FQdTvhtvdhP58oumbexLP
PR LintLabelPR must have at least one of these labels: TitlePR title must follow conventional commit format: Allowed types: Example: |
Contributor
📝 WalkthroughWalkthroughThe English and German localization files replace “registered” wording with “applied” wording across assignment, status, supervisor, delegation, conference, statistics, and participant messages. ChangesApplication terminology localization
Estimated code review effort: 1 (Trivial) | ~3 minutes 🚥 Pre-merge checks | ✅ 7 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (7 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@messages/de.json`:
- Line 553: Update the German text in delegationsDescription to use the
gender-inclusive form UN-Generalsekretär*innen instead of the masculine-only
UN-Generalsekretäre, leaving the rest of the message unchanged.
- Line 42: In the adminAssignmentDescription translation, correct the German
spelling by replacing “durck” with “durch” while leaving the rest of the
description unchanged.
- Line 171: Update the translation value for
assistantFlowSupervisorUsersAlreadyRegistered2 so the final phrase refers to
adding the supervisor as “Betreuer*in” rather than adding supervision as
“Betreuung”. Preserve the rest of the German message unchanged.
In `@messages/en.json`:
- Line 205: Update the averageAgeOnlyApplied translation in messages/en.json at
lines 205-205 to use “Average age at conference (applicants only)”, and update
the corresponding translation in messages/de.json at lines 205-205 to use
“Konferenz-Altersdurchschnitt (nur Bewerbende)”.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 3404cda8-bf1f-4011-901d-868359b5bf72
📒 Files selected for processing (2)
messages/de.jsonmessages/en.json
| "adminActions": "Aktionen", | ||
| "adminAssignment": "Zuteilung", | ||
| "adminAssignmentDescription": "Dieses Tool führt dich interaktiv durck die Zuweisung von Angemeldeten Teilnehmenden zu Delegationen. Ein Algorithmus macht dabei Vorschläge, wie die Verteilung am besten aufgeht.<br><strong>Laden Sie zunächst die Ergebnisse als JSON-Datei herunter und starten Sie dann das Tool. Sie werden aufgefordert, die gerade heruntergeladene Datei hochzuladen.</strong>", | ||
| "adminAssignmentDescription": "Dieses Tool führt dich interaktiv durck die Zuweisung von beworbenen Teilnehmenden zu Delegationen. Ein Algorithmus macht dabei Vorschläge, wie die Verteilung am besten aufgeht.<br><strong>Laden Sie zunächst die Ergebnisse als JSON-Datei herunter und starten Sie dann das Tool. Sie werden aufgefordert, die gerade heruntergeladene Datei hochzuladen.</strong>", |
Contributor
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Fix the spelling error in the assignment description.
Replace durck with durch.
Suggested fix
- "Dieses Tool führt dich interaktiv durck die Zuweisung ..."
+ "Dieses Tool führt dich interaktiv durch die Zuweisung ..."📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "adminAssignmentDescription": "Dieses Tool führt dich interaktiv durck die Zuweisung von beworbenen Teilnehmenden zu Delegationen. Ein Algorithmus macht dabei Vorschläge, wie die Verteilung am besten aufgeht.<br><strong>Laden Sie zunächst die Ergebnisse als JSON-Datei herunter und starten Sie dann das Tool. Sie werden aufgefordert, die gerade heruntergeladene Datei hochzuladen.</strong>", | |
| "adminAssignmentDescription": "Dieses Tool führt dich interaktiv durch die Zuweisung von beworbenen Teilnehmenden zu Delegationen. Ein Algorithmus macht dabei Vorschläge, wie die Verteilung am besten aufgeht.<br><strong>Laden Sie zunächst die Ergebnisse als JSON-Datei herunter und starten Sie dann das Tool. Sie werden aufgefordert, die gerade heruntergeladene Datei hochzuladen.</strong>", |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@messages/de.json` at line 42, In the adminAssignmentDescription translation,
correct the German spelling by replacing “durck” with “durch” while leaving the
rest of the description unchanged.
| "assistantFlowRolePress3": "Du schreibst Artikel für die Printzeitung oder produzierst die Fernsehnachrichten, die an den Konferenzabenden gezeigt werden.", | ||
| "assistantFlowSupervisorUsersAlreadyRegistered1": "Sehr schön! Als Betreuer*in kannst du die von deinen Schüler*innen erstellten Delegationen und Einzeldelegierte beaufsichtigen.", | ||
| "assistantFlowSupervisorUsersAlreadyRegistered2": "Wichtig: Anders als früher kannst du als Betreuer*in keine Delegation mehr für deine Schüler*innen anlegen, sondern die Schüler*innen müssen ihre Delegation selbst erstellen! Wenn du dich als Betreuer*in angemeldet hast, kannst du mit deinen Schüler*innen dann einen Beitrittscode teilen, mit dem sie dich als Betreuung hinzufügen.\n", | ||
| "assistantFlowSupervisorUsersAlreadyRegistered2": "Wichtig: Anders als früher kannst du als Betreuer*in keine Delegation mehr für deine Schüler*innen anlegen, sondern die Schüler*innen müssen ihre Delegation selbst erstellen! Wenn du dich als Betreuer*in beworben hast, kannst du mit deinen Schüler*innen dann einen Beitrittscode teilen, mit dem sie dich als Betreuung hinzufügen.\n", |
Contributor
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Refer to the supervisor as a person.
als Betreuung hinzufügen means to add supervision, not a supervisor. Use als Betreuer*in hinzufügen.
Suggested fix
- "... mit dem sie dich als Betreuung hinzufügen."
+ "... mit dem sie dich als Betreuer*in hinzufügen."📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "assistantFlowSupervisorUsersAlreadyRegistered2": "Wichtig: Anders als früher kannst du als Betreuer*in keine Delegation mehr für deine Schüler*innen anlegen, sondern die Schüler*innen müssen ihre Delegation selbst erstellen! Wenn du dich als Betreuer*in beworben hast, kannst du mit deinen Schüler*innen dann einen Beitrittscode teilen, mit dem sie dich als Betreuung hinzufügen.\n", | |
| "assistantFlowSupervisorUsersAlreadyRegistered2": "Wichtig: Anders als früher kannst du als Betreuer*in keine Delegation mehr für deine Schüler*innen anlegen, sondern die Schüler*innen müssen ihre Delegation selbst erstellen! Wenn du dich als Betreuer*in beworben hast, kannst du mit deinen Schüler*innen dann einen Beitrittscode teilen, mit dem sie dich als Betreuer*in hinzufügen.\n", |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@messages/de.json` at line 171, Update the translation value for
assistantFlowSupervisorUsersAlreadyRegistered2 so the final phrase refers to
adding the supervisor as “Betreuer*in” rather than adding supervision as
“Betreuung”. Preserve the rest of the German message unchanged.
| "delegationStatusDescription": "Dein zugewiesenes Land oder deine Nichtstaatliche Organisation und Delegationsstatistiken.", | ||
| "delegations": "Delegationen", | ||
| "delegationsDescription": "Hier findest du alle Delegationen, in denen Schüler*innen von dir angemeldet sind. Jede Delegation hat einen generierten Namen mit einem der UN-Generalsekretäre - gib diesen Namen an, wenn du mit der Teilnehmendenbetreuung sprichst. Delegationsmitglieder, die dich nicht als Betreuer*in hinzugefügt haben, sind als \"versteckte Mitglieder\" gekennzeichnet.", | ||
| "delegationsDescription": "Hier findest du alle Delegationen, in denen sich Schüler*innen von dir beworben haben. Jede Delegation hat einen generierten Namen mit einem der UN-Generalsekretäre - gib diesen Namen an, wenn du mit der Teilnehmendenbetreuung sprichst. Delegationsmitglieder, die dich nicht als Betreuer*in hinzugefügt haben, sind als \"versteckte Mitglieder\" gekennzeichnet.", |
Contributor
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Use gender-inclusive language for the office holder.
UN-Generalsekretäre is a masculine-only person noun. Replace it with UN-Generalsekretär*innen.
As per coding guidelines, person-related nouns in messages/de.json must use gender-inclusive language with gender star (*) or neutral participle forms.
Suggested fix
- "... mit einem der UN-Generalsekretäre ..."
+ "... mit einem der UN-Generalsekretär*innen ..."📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "delegationsDescription": "Hier findest du alle Delegationen, in denen sich Schüler*innen von dir beworben haben. Jede Delegation hat einen generierten Namen mit einem der UN-Generalsekretäre - gib diesen Namen an, wenn du mit der Teilnehmendenbetreuung sprichst. Delegationsmitglieder, die dich nicht als Betreuer*in hinzugefügt haben, sind als \"versteckte Mitglieder\" gekennzeichnet.", | |
| "delegationsDescription": "Hier findest du alle Delegationen, in denen sich Schüler*innen von dir beworben haben. Jede Delegation hat einen generierten Namen mit einem der UN-Generalsekretär*innen - gib diesen Namen an, wenn du mit der Teilnehmendenbetreuung sprichst. Delegationsmitglieder, die dich nicht als Betreuer*in hinzugefügt haben, sind als \"versteckte Mitglieder\" gekennzeichnet.", |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@messages/de.json` at line 553, Update the German text in
delegationsDescription to use the gender-inclusive form UN-Generalsekretär*innen
instead of the masculine-only UN-Generalsekretäre, leaving the rest of the
message unchanged.
Source: Coding guidelines
| "author": "Author", | ||
| "authorization": "Authorization", | ||
| "averageAgeOnlyApplied": "Average age at conference (registered only)", | ||
| "averageAgeOnlyApplied": "Average age at conference (applied only)", |
Contributor
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Use explicit applicant labels in both locales.
The current labels do not clearly identify the applicant population.
messages/en.json#L205-L205: useAverage age at conference (applicants only).messages/de.json#L205-L205: useKonferenz-Altersdurchschnitt (nur Bewerbende).
📍 Affects 2 files
messages/en.json#L205-L205(this comment)messages/de.json#L205-L205
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@messages/en.json` at line 205, Update the averageAgeOnlyApplied translation
in messages/en.json at lines 205-205 to use “Average age at conference
(applicants only)”, and update the corresponding translation in messages/de.json
at lines 205-205 to use “Konferenz-Altersdurchschnitt (nur Bewerbende)”.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR updates translation strings across German and English locales to clarify the distinction between "applied" (beworden/applied) and "registered" (angemeldet/registered) in the context of conference participation. The changes reflect that users first apply to a conference, and only after acceptance are they registered.
Key Changes
alreadRegistered,applied,notApplied)Notable Implementation Details
messages/de.jsonandmessages/en.jsonto maintain parityhttps://claude.ai/code/session_018FQdTvhtvdhP58oumbexLP
Summary by CodeRabbit