Skip to content

fix: settingsLoading -> loading#750

Open
lc-soft wants to merge 1 commit intoRightNow-AI:mainfrom
lc-soft:fix/settings-page-error
Open

fix: settingsLoading -> loading#750
lc-soft wants to merge 1 commit intoRightNow-AI:mainfrom
lc-soft:fix/settings-page-error

Conversation

@lc-soft
Copy link

@lc-soft lc-soft commented Mar 19, 2026

Summary

This PR fixes an Alpine.js runtime error caused by a typo in the variable name within a conditional expression.

screenshot-20260319-192847

Changes

Corrected the typo in the Alpine expression from !settignsLoading to !loading.

Testing

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Live integration tested (if applicable)

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • User input validated at boundaries

Copy link
Member

@jaberjaber23 jaberjaber23 left a comment

Choose a reason for hiding this comment

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

Clean fix. \ doesn't exist in the component data — \ is the correct property name. Two-line fix, verified against settings.js.

Copy link
Member

@jaberjaber23 jaberjaber23 left a comment

Choose a reason for hiding this comment

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

Clean fix. settingsLoading doesn't exist in the component data — loading is the correct property name. Two-line fix, verified against settings.js.

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