Skip to content

Fix settings export filename assignment#215

Merged
bartekplus merged 1 commit intomasterfrom
codex/fix-settings-export-filename
Feb 24, 2026
Merged

Fix settings export filename assignment#215
bartekplus merged 1 commit intomasterfrom
codex/fix-settings-export-filename

Conversation

@bartekplus
Copy link
Owner

@bartekplus bartekplus commented Feb 24, 2026

Summary

  • remove redundant dlink.download = name assignment in settings export flow
  • assign export filename once via explicit exportFilename constant
  • replace comma-expression assignment with clear statement-based handler setup

Validation

  • node --check src/third_party/fancier-settings/settings.js

@bartekplus bartekplus merged commit d1b46a5 into master Feb 24, 2026
4 checks passed
@bartekplus bartekplus deleted the codex/fix-settings-export-filename branch February 24, 2026 16:54
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