Skip to content

refactor: sort selected languages in language edit component for impr…#122

Merged
timosville merged 5 commits into
mainfrom
sco-2488-language-listing-system
Jun 4, 2026
Merged

refactor: sort selected languages in language edit component for impr…#122
timosville merged 5 commits into
mainfrom
sco-2488-language-listing-system

Conversation

@timosville

Copy link
Copy Markdown
Contributor

No description provided.

@timosville timosville requested a review from JannieT June 4, 2026 07:59
@linear-code

linear-code Bot commented Jun 4, 2026

Copy link
Copy Markdown

SCO-2488

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 4, 2026

Copy link
Copy Markdown

Deploying kit with  Cloudflare Pages  Cloudflare Pages

Latest commit: e090efe
Status:⚡️  Build in progress...

View logs

@JannieT JannieT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! Just relocate the logic closer to where it is used.

Comment thread src/frontend/shared/helpers.ts Outdated

@JannieT JannieT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. Merge when you have relocated the three functions.

@timosville timosville merged commit 22c5235 into main Jun 4, 2026
2 of 3 checks passed
@timosville timosville deleted the sco-2488-language-listing-system branch June 4, 2026 10:41
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