Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: export Locale as named exports #219

Merged
merged 1 commit into from
Feb 20, 2024
Merged

Conversation

ghiscoding
Copy link
Owner

@ghiscoding ghiscoding commented Feb 20, 2024

  • allow to import a single Locale via named imports
  • for example:
import { Spanish } from 'multiple-select-vanilla/dist/locales/multiple-select-es-ES';

multipleSelect('#select', { locale: Spanish });
  • also restrict the acceptable Locale key that can be provided to locale, for example:
    • { locale: 'ja-JP' } is valid
    • { locale: 'unknown' } is invalid

- allow to import a single Locale via named imports
- for example:
```ts
import { Spanish } from 'multiple-select-vanilla/dist/locales/multiple-select-es-ES';

multipleSelect('#select', { locale: Spanish });
```
- also restrict the acceptable Locale key that can be provided to `locale` (e.g.: `{ locale: 'ja-JP' }`, but `locale: 'unknown' would be invalid)
Copy link

Playwright E2E Test Results

71 tests  +1   71 ✅ +1   2m 2s ⏱️ +10s
63 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 8d47c16. ± Comparison against base commit 38d3142.

@ghiscoding ghiscoding merged commit 6eb57ce into main Feb 20, 2024
3 checks passed
@ghiscoding ghiscoding deleted the feat/named-locale-imports branch February 20, 2024 02:18
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