Skip to content

Fix: merge speakers typos - #1076

Merged
ako3131 merged 1 commit into
masterfrom
fix/merge-speakers-typos
Oct 23, 2026
Merged

ako3131 merged 1 commit into
masterfrom
fix/merge-speakers-typos

Conversation

@ako3131

@ako3131 ako3131 commented Sep 18, 2026 •

Copy link
Copy Markdown

ref: https://app.clickup.com/t/9014802374/86bc38e6w

Bug Fixes
-Fixed a typo: "Laguages" → "Languages".
-Expanded abbreviated field labels that were hard to read: "Reg. Email" → "Registration Email", "Other Prez" → "Other Presentation Links", "Travel Pref." → "Travel Preferences", "Org. Roles" → "Organizational Roles".

@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8654d5f2-fcd5-4438-8661-030603dd3507
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@ako3131
ako3131 requested a review from smarcet September 22, 2026 19:46
@ako3131 ako3131 self-assigned this Sep 22, 2026
@ako3131 ako3131 closed this Sep 24, 2026
@ako3131
ako3131 removed the request for review from smarcet September 29, 2026 23:34
@ako3131

ako3131 commented Sep 29, 2026

Copy link
Copy Markdown
Author

/deploy-preview

@ako3131 ako3131 reopened this Sep 29, 2026
@github-actions

Copy link
Copy Markdown

🚀 Deployed on https://pr-1076--show-admin-preview.netlify.app

@ako3131 ako3131 closed this Sep 29, 2026
@ako3131 ako3131 reopened this Sep 30, 2026
@ako3131
ako3131 force-pushed the fix/merge-speakers-typos branch from f41d397 to dd0802d Compare October 22, 2026 17:19
@ako3131
ako3131 requested a balanced review from Copilot October 22, 2026 17:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

All five label changes match the stated scope, preserve existing translation keys, and introduce no identified issues.

No open issues 🎉

What changed in this PR

Improves the English field labels in the speaker merge interface without changing translation keys or application logic.

Changes:

  • Corrects “Laguages” to “Languages”.
  • Expands four abbreviated labels for clarity.
File Description
src/​i18n/​en.json Corrects one typo and expands four speaker merge field labels.

Review effort: Balanced

@santipalenque santipalenque left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@ako3131
ako3131 merged commit 52a0cca into master Oct 23, 2026
10 checks passed
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.

3 participants