Skip to content

Allow individual import failures and provide status updates#920

Merged
killergerbah merged 1 commit intomainfrom
import-allow-fail
Mar 4, 2026
Merged

Allow individual import failures and provide status updates#920
killergerbah merged 1 commit intomainfrom
import-allow-fail

Conversation

@ShanaryS
Copy link
Copy Markdown
Collaborator

@ShanaryS ShanaryS commented Mar 4, 2026

If a single entry from the import failed (e.g could not determine lemma) then it would cancel the entire import. This would be fine for asbplayer export-import but we should just do a best-effort import instead.

@ShanaryS ShanaryS self-assigned this Mar 4, 2026
@ShanaryS ShanaryS added the bug Something isn't working label Mar 4, 2026
@ShanaryS ShanaryS force-pushed the import-allow-fail branch from d5b04cb to 92efed3 Compare March 4, 2026 05:33
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Mar 4, 2026

Deploying asbplayer with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3c53c05
Status: ✅  Deploy successful!
Preview URL: https://5ff8922b.asbplayer.pages.dev
Branch Preview URL: https://import-allow-fail.asbplayer.pages.dev

View logs

@ShanaryS ShanaryS force-pushed the import-allow-fail branch from 92efed3 to 3c53c05 Compare March 4, 2026 05:39
@killergerbah killergerbah merged commit 6b693a3 into main Mar 4, 2026
2 checks passed
@killergerbah killergerbah deleted the import-allow-fail branch March 4, 2026 12:55
@killergerbah killergerbah added this to the Extension v1.15.0 milestone Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants