Skip to content

Adding tamil javascript sdk#177

Closed
navidik wants to merge 3 commits intoasgardeo:mainfrom
navidik:adding-tamil-javascript-sdk
Closed

Adding tamil javascript sdk#177
navidik wants to merge 3 commits intoasgardeo:mainfrom
navidik:adding-tamil-javascript-sdk

Conversation

@navidik
Copy link
Contributor

@navidik navidik commented Sep 27, 2025

Purpose

Related Issues

Related PRs

Checklist

  • e2e cypress tests locally verified.
  • Manual test round performed and verified.
  • UX/UI review done on the final implementation.
  • Documentation provided. (Add links if there are any)
  • Unit tests provided. (Add links if there are any)
  • Integration tests provided. (Add links if there are any)

Security checks

<Add Tamil (ta-IN) language support to the asgardeo>

<Create a new locale file:packages/i18n/src/locales/ta-IN.ts + Add Tamil translations for all existing UI components>

#<164>
@brionmario
Copy link
Member

brionmario commented Sep 30, 2025

Hey @navidik,

Thanks a lot for the PR 🙌

Inorder for me to merge the PR, lets do the following changes:

Screenshot 2025-09-30 at 09 24 48

Copy link
Member

Choose a reason for hiding this comment

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

@navidik

Shall we revert the changes to this file?

@brionmario
Copy link
Member

brionmario commented Oct 1, 2025

@navidik Shall we take a pull and move the file to the new location.

The guide can be found here: https://github.com/asgardeo/javascript/blob/main/CONTRIBUTING.md#contributing-to-the-internalization-i18n-package.

And lets update the PR description as well with a proper description, etc.
Refer the issue you were assigned in Related Issues section:

Ex:

### Related Issues
 - Fixes <ISSUE_LINK>

@navidik navidik closed this Oct 2, 2025
@navidik navidik deleted the adding-tamil-javascript-sdk branch October 2, 2025 11:44
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.

2 participants