Skip to content

Conversation

JosephSanjaya
Copy link
Collaborator

📱 Description

The Indonesian language code "in" is normalized to "id" in MessagingTopicDataSource to align with the ISO 639-1 standard used by the backend.

This ensures consistency in topic generation for Indonesian language users. Tests have been added to verify this normalization for both "in" and "in_ID" inputs.

Platform

  • Android
  • iOS
  • Games-Unity
  • DevOps (AWS)
  • Website
  • C/Golang

🎯 Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Refactoring (code change that neither fixes a bug nor adds a feature)
  • 📚 Documentation update
  • 🎨 UI/UX improvement
  • ⚡ Performance improvement
  • 🧪 Test addition or improvement

📋 Changes

  • Add Normalization for id

🔗 Related Issues

🧪 Tests Status

  • Tests ran successfully locally?
  • Added more tests? How many? Added 1 test to verify always subscribed to id not in
  • Code coverage percentage of the codebase: __%

📸 Screenshots/Videos

N/A Behavior is not changing on front side

🎯 Reviewers

@kcw-grunt, @josikie

The Indonesian language code "in" is normalized to "id" in `MessagingTopicDataSource`
to align with the ISO 639-1 standard used by the backend.

This ensures consistency in topic generation for Indonesian language users.
Tests have been added to verify this normalization for both "in" and "in_ID" inputs.
@JosephSanjaya JosephSanjaya force-pushed the js/chore/#234-topic-normalization branch from fbcc782 to 22125fd Compare August 30, 2025 10:26
@JosephSanjaya
Copy link
Collaborator Author

rebased

Copy link
Collaborator

@kcw-grunt kcw-grunt left a comment

Choose a reason for hiding this comment

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

LGTM

} ?: defaultLanguage

return targetLanguage.code.split("-").first().lowercase()
val baseCode = targetLanguage.code.split("-").first().lowercase()
Copy link
Collaborator

Choose a reason for hiding this comment

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

🎉

@kcw-grunt kcw-grunt added techdebt Fixes that maintain the code android labels Aug 30, 2025
@kcw-grunt kcw-grunt merged commit 9f4908b into develop Aug 30, 2025
2 checks passed
@kcw-grunt kcw-grunt deleted the js/chore/#234-topic-normalization branch August 30, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android techdebt Fixes that maintain the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants